Merge branch 'release9.2/fixes/modify-data-assert' into v9.2_patches

dyninst · Jul 11, 2016 · 80a0838 · 80a0838
2 parents 3a6ad66 + d0b8e3d
commit 80a0838
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/dyninstAPI/src/codegen-x86.C b/dyninstAPI/src/codegen-x86.C
@@ -1203,7 +1203,7 @@ bool insnCodeGen::modifyData(Address targetAddr, instruction &insn, codeGen &gen
     origInsn += pref_count;
 
     /* Decode the opcode */
-    if(ia32_decode_opcode(0, origInsn, instruct, NULL))
+    if(ia32_decode_opcode(0, origInsn, instruct, NULL) < 0)
         assert(!"Couldn't decode opcode of already known instruction!\n");
 
     /* Calculate the amount of opcode bytes */
@@ -1328,7 +1328,7 @@ bool insnCodeGen::modifyDisp(signed long newDisp, instruction &insn, codeGen &ge
     origInsn += pref_count;
 
     /* Decode the opcode */
-    if(ia32_decode_opcode(0, origInsn, instruct, NULL))
+    if(ia32_decode_opcode(0, origInsn, instruct, NULL) < 0)
         assert(!"Couldn't decode opcode of already known instruction!\n");
 
     /* Calculate the amount of opcode bytes */