Skip to content

Commit

Permalink
Change byte[][] to byte[][2]
Browse files Browse the repository at this point in the history
  • Loading branch information
ltyu committed Nov 2, 2024
1 parent 79b4f8c commit 36a9d1f
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 2 deletions.
2 changes: 1 addition & 1 deletion solidity/contracts/isms/ccip-read/StorageProofIsm.sol
Original file line number Diff line number Diff line change
Expand Up @@ -116,7 +116,7 @@ abstract contract StorageProofIsm is AbstractCcipReadIsm, OwnableUpgradeable {
bytes32 _dispatchedSlotKey
) public view returns (bytes memory) {
// Get the slot value as bytes
bytes[][] memory proofs = abi.decode(_proofs, (bytes[][]));
bytes[][2] memory proofs = abi.decode(_proofs, (bytes[][2]));

Check notice

Code scanning / Olympix Integrated Security

Local variables in test functions are not properly fuzzed, potentially reducing the effectiveness of property-based testing. For more information, visit: http://detectors.olympixdevsectools.com/article/web3-vulnerability/unfuzzed-local-variables Low

Local variables in test functions are not properly fuzzed, potentially reducing the effectiveness of property-based testing. For more information, visit: http://detectors.olympixdevsectools.com/article/web3-vulnerability/unfuzzed-local-variables
bytes[] memory accountProof = proofs[0];

Check notice

Code scanning / Olympix Integrated Security

Local variables in test functions are not properly fuzzed, potentially reducing the effectiveness of property-based testing. For more information, visit: http://detectors.olympixdevsectools.com/article/web3-vulnerability/unfuzzed-local-variables Low

Local variables in test functions are not properly fuzzed, potentially reducing the effectiveness of property-based testing. For more information, visit: http://detectors.olympixdevsectools.com/article/web3-vulnerability/unfuzzed-local-variables
bytes[] memory storageProof = proofs[1];

Check notice

Code scanning / Olympix Integrated Security

Local variables in test functions are not properly fuzzed, potentially reducing the effectiveness of property-based testing. For more information, visit: http://detectors.olympixdevsectools.com/article/web3-vulnerability/unfuzzed-local-variables Low

Local variables in test functions are not properly fuzzed, potentially reducing the effectiveness of property-based testing. For more information, visit: http://detectors.olympixdevsectools.com/article/web3-vulnerability/unfuzzed-local-variables

Expand Down
2 changes: 1 addition & 1 deletion solidity/test/isms/Sp1LightClientIsm.t.sol
Original file line number Diff line number Diff line change
Expand Up @@ -100,7 +100,7 @@ contract SP1LightClientIsmTest is StateProofHelpersTest {
}

function _encodeProofs() internal view returns (bytes memory) {
return abi.encode(accountProof, storageProof);
return abi.encode([accountProof, storageProof]);
}

function testSP1LightClientIsm_setOffchainUrls_revertsWithNonOwner(
Expand Down

0 comments on commit 36a9d1f

Please sign in to comment.