GUI: fixed a bug with copying the wrong number of instructions (thanks to futureproof for the find!)

mrexodia · mrexodia · commit efe2f1221942 · 2014-12-29T20:21:51.000+01:00
diff --git a/x64_dbg_gui/Project/Src/BasicView/Disassembly.cpp b/x64_dbg_gui/Project/Src/BasicView/Disassembly.cpp
@@ -1175,13 +1175,11 @@ void Disassembly::prepareDataRange(int_t startRva, int_t endRva, QList<Instructi
     {
         int wCount = 0;
         int_t addr = startRva;
-        while(addr < endRva)
+        while(addr <= endRva)
         {
             addr = getNextInstructionRVA(addr, 1);
             wCount++;
         }
-        if(addr - 1 != endRva)
-            wCount--;
         prepareDataCount(startRva, wCount, instBuffer);
     }
 }

Original file line number	Diff line number	Diff line change
`@@ -1175,13 +1175,11 @@ void Disassembly::prepareDataRange(int_t startRva, int_t endRva, QList<Instructi`
`1175`	`1175`	`{`
`1176`	`1176`	`int wCount = 0;`
`1177`	`1177`	`int_t addr = startRva;`
`1178`		`- while(addr < endRva)`
	`1178`	`+ while(addr <= endRva)`
`1179`	`1179`	`{`
`1180`	`1180`	`addr = getNextInstructionRVA(addr, 1);`
`1181`	`1181`	`wCount++;`
`1182`	`1182`	`}`
`1183`		`- if(addr - 1 != endRva)`
`1184`		`- wCount--;`
`1185`	`1183`	`prepareDataCount(startRva, wCount, instBuffer);`
`1186`	`1184`	`}`
`1187`	`1185`	`}`