Skip to content

Commit

Permalink
[#754] Update comment on fixed signature part
Browse files Browse the repository at this point in the history
  • Loading branch information
akshay-ap committed Jul 11, 2024
1 parent 2743e53 commit 025234f
Showing 1 changed file with 2 additions and 1 deletion.
3 changes: 2 additions & 1 deletion modules/4337/contracts/Safe4337Module.sol
Original file line number Diff line number Diff line change
Expand Up @@ -228,7 +228,8 @@ contract Safe4337Module is IAccount, HandlerContext, CompatibilityFallbackHandle
/// @solidity memory-safe-assembly
assembly {
let signaturePos := mul(0x41, i)
// read the Safe signature type byte from the signatures bytes, this is the 64th byte per signature.
// read the Safe signature type byte from the signatures bytes, this is the 65th byte per signature.
// The fixed part of the signature is encoded as {32-bytes signature verifier}{32-bytes dynamic data position}{1-byte signature type}
let signatureType := byte(0, calldataload(add(signatures.offset, add(signaturePos, 0x40))))

// signatureType = 0 indicates that signature is a smart contract signature in Safe Signature Encoding
Expand Down

0 comments on commit 025234f

Please sign in to comment.