Commit a4fae7c7 authored by Kelvin Fichter's avatar Kelvin Fichter Committed by GitHub

Update OVM_L1CrossDomainMessenger (#76)

* Update OVM_L1CrossDomainMessenger

Fixes problem in the value we were verifying.

* Fix broken tests
Co-authored-by: default avatarKarl Floersch <karl@karlfloersch.com>
parent 0125c1ec
...@@ -237,7 +237,7 @@ contract OVM_L1CrossDomainMessenger is iOVM_L1CrossDomainMessenger, OVM_BaseCros ...@@ -237,7 +237,7 @@ contract OVM_L1CrossDomainMessenger is iOVM_L1CrossDomainMessenger, OVM_BaseCros
return Lib_SecureMerkleTrie.verifyInclusionProof( return Lib_SecureMerkleTrie.verifyInclusionProof(
abi.encodePacked(storageKey), abi.encodePacked(storageKey),
abi.encodePacked(uint256(1)), abi.encodePacked(uint8(1)),
_proof.storageTrieWitness, _proof.storageTrieWitness,
account.storageRoot account.storageRoot
); );
......
...@@ -198,7 +198,7 @@ describe('OVM_L1CrossDomainMessenger', () => { ...@@ -198,7 +198,7 @@ describe('OVM_L1CrossDomainMessenger', () => {
nodes: [ nodes: [
{ {
key: storageKey, key: storageKey,
val: '0x' + '01'.padStart(64, '0'), val: '0x' + '01'.padStart(2, '0'),
}, },
], ],
secure: true, secure: true,
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment