Update OVM_L1CrossDomainMessenger (#76)

* Update OVM_L1CrossDomainMessenger Fixes problem in the value we were verifying. * Fix broken tests Co-authored-by: Karl Floersch <karl@karlfloersch.com>
ethereum-optimism · Nov 23, 2020 · a4fae7c · a4fae7c
1 parent 0125c1e
commit a4fae7c
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/packages/contracts/contracts/optimistic-ethereum/OVM/bridge/OVM_L1CrossDomainMessenger.sol b/packages/contracts/contracts/optimistic-ethereum/OVM/bridge/OVM_L1CrossDomainMessenger.sol
@@ -237,7 +237,7 @@ contract OVM_L1CrossDomainMessenger is iOVM_L1CrossDomainMessenger, OVM_BaseCros
 
         return Lib_SecureMerkleTrie.verifyInclusionProof(
             abi.encodePacked(storageKey),
-            abi.encodePacked(uint256(1)),
+            abi.encodePacked(uint8(1)),
             _proof.storageTrieWitness,
             account.storageRoot
         );

diff --git a/packages/contracts/test/contracts/OVM/bridge/OVM_L1CrossDomainMessenger.spec.ts b/packages/contracts/test/contracts/OVM/bridge/OVM_L1CrossDomainMessenger.spec.ts
@@ -198,7 +198,7 @@ describe('OVM_L1CrossDomainMessenger', () => {
         nodes: [
           {
             key: storageKey,
-            val: '0x' + '01'.padStart(64, '0'),
+            val: '0x' + '01'.padStart(2, '0'),
           },
         ],
         secure: true,