Merge pull request #84 from astewart-bah/master

Update mem_changer.py to address logic bug in add_to_mem.
angr · Feb 14, 2024 · ace8419 · ace8419
2 parents f2e4616 + cfd6775
commit ace8419
Showing 1 changed file with 4 additions and 4 deletions.
diff --git a/angrop/chain_builder/mem_changer.py b/angrop/chain_builder/mem_changer.py
@@ -115,13 +115,13 @@ def add_to_mem(self, addr, value, data_size=None):
         l.debug("Now building the mem add chain")
 
         # try to build the chain
-        try:
-            for g in gadgets:
+        for g in gadgets:
+            try:
                 chain = self._add_mem_with_gadget(g, addr, data_size, difference=value)
                 self.verify(chain, addr, value, data_size)
                 return chain
-        except RopException:
-            pass
+            except RopException:
+                pass
 
         raise RopException("Fail to perform add_to_mem!")