From c8702f686c8f5e64baf900f62fdf1cc3fe9f1330 Mon Sep 17 00:00:00 2001 From: Akshay Date: Mon, 8 Jul 2024 13:58:38 +0200 Subject: [PATCH] [#754] Fix lint issue --- modules/4337/contracts/Safe4337Module.sol | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/modules/4337/contracts/Safe4337Module.sol b/modules/4337/contracts/Safe4337Module.sol index d8b6f886..12142f77 100644 --- a/modules/4337/contracts/Safe4337Module.sol +++ b/modules/4337/contracts/Safe4337Module.sol @@ -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