Skip to content

Commit

Permalink
fix: [ALC-LA2-1] document internal function _getCombinedSalt to expec…
Browse files Browse the repository at this point in the history
…t clean upper bits for owner
  • Loading branch information
jaypaik committed Apr 5, 2024
1 parent 524ee49 commit f2fd387
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 1 deletion.
1 change: 1 addition & 0 deletions src/LightAccountFactory.sol
Original file line number Diff line number Diff line change
Expand Up @@ -50,6 +50,7 @@ contract LightAccountFactory is BaseLightAccountFactory {
}

/// @notice Compute the hash of the owner and salt in scratch space memory.
/// @dev The caller is responsible for cleaning the upper bits of the owner address parameter.
/// @param owner The owner of the account to be created.
/// @param salt A salt, which can be changed to create multiple accounts with the same owner.
/// @return combinedSalt The hash of the owner and salt.
Expand Down
2 changes: 1 addition & 1 deletion test/LightAccount.t.sol
Original file line number Diff line number Diff line change
Expand Up @@ -556,7 +556,7 @@ contract LightAccountTest is Test {
bytes32(uint256(uint160(0x0000000071727De22E5E9d8BAf0edAc6f37da032)))
)
),
0xefe804f2c00bf5ade462dc2ba6c9c59fe5a69a24914737fbd7613bdb2dfcf600
0xd163ff9f4127df29ad58e4686d0f5367607e32fe777a91a8f4d86fdab6e23640
);
}

Expand Down

0 comments on commit f2fd387

Please sign in to comment.