Skip to content

Commit

Permalink
Merge pull request #184 from Consensys/fix/target-mimcHash-bytes32
Browse files Browse the repository at this point in the history
fix: target type to bytes32 instead of bytes to adapt to new mimc hash
  • Loading branch information
Julink-eth authored Jul 5, 2024
2 parents e61c515 + fd7758a commit 68f4e80
Showing 1 changed file with 1 addition and 3 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -138,9 +138,7 @@ library LineaProofHelper {
bytes32 stateRoot
) private pure returns (bool) {
// Verify the target contract first against the account proof's last leaf node's hkey
bytes32 targetHash = SparseMerkleProof.mimcHash(
abi.encodePacked(target)
);
bytes32 targetHash = SparseMerkleProof.mimcHash(abi.encode(target));
SparseMerkleProof.Leaf memory accountLeaf = SparseMerkleProof.getLeaf(
accountProof.proof.proofRelatedNodes[LAST_LEAF_INDEX]
);
Expand Down

0 comments on commit 68f4e80

Please sign in to comment.