Skip to content

Commit

Permalink
[#754] Fix lint issue
Browse files Browse the repository at this point in the history
  • Loading branch information
akshay-ap committed Jul 8, 2024
1 parent 9c131c6 commit c8702f6
Showing 1 changed file with 1 addition and 2 deletions.
3 changes: 1 addition & 2 deletions modules/4337/contracts/Safe4337Module.sol
Original file line number Diff line number Diff line change
Expand Up @@ -266,8 +266,7 @@ contract Safe4337Module is IAccount, HandlerContext, CompatibilityFallbackHandle
}

// checkSignatures function in Safe contract does not force a fixed size on signature length. A malicious actor can append additional bytes in the signature than needed and force gas usage to reach verificationGasLimit. _checkSignatureLength ensures that there are no additional bytes in the signatures than required.
validSignature = validSignature
&& _checkSignatureLength(signatures, ISafe(payable(userOp.sender)).getThreshold());
validSignature = validSignature && _checkSignatureLength(signatures, ISafe(payable(userOp.sender)).getThreshold());

if (validSignature) {
// The timestamps are validated by the entry point, therefore we will not check them again
Expand Down

0 comments on commit c8702f6

Please sign in to comment.