Merge pull request #684 from g-r-a-n-t/bugfix

fixed memory bug in storage maps
ethereum · Mar 8, 2022 · d92cd20 · d92cd20
2 parents a86a9fb + 16ea2a9
commit d92cd20
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 1 deletion.
diff --git a/crates/yulgen/src/runtime/functions/data.rs b/crates/yulgen/src/runtime/functions/data.rs
@@ -350,7 +350,7 @@ pub fn alloc_mstoren() -> yul::Statement {
 pub fn map_value_ptr() -> yul::Statement {
     function_definition! {
         function map_value_ptr(a, b) -> return_val {
-            (let ptr := avail())
+            (let ptr := alloc(64))
             (mstore(ptr, a))
             (mstore((add(ptr, 32)), b))
             (let hash := keccak256(ptr, 64))

diff --git a/newsfragments/684.bugfix.md b/newsfragments/684.bugfix.md
@@ -0,0 +1 @@
+The region of memory used to compute the slot of a storage map value was not being allocated.