fix: use msg.sender instead of userOp.sender

rhinestonewtf · Apr 4, 2024 · ec3760e · ec3760e
1 parent 579ea0b
commit ec3760e
Showing 1 changed file with 3 additions and 3 deletions.
diff --git a/accounts/safe7579/src/SafeERC7579.sol b/accounts/safe7579/src/SafeERC7579.sol
@@ -174,7 +174,7 @@ contract SafeERC7579 is
         // pay prefund
         if (missingAccountFunds != 0) {
             _execute({
-                safe: userOp.getSender(),
+                safe: msg.sender,
                 target: entryPoint(),
                 value: missingAccountFunds,
                 callData: ""
@@ -197,7 +197,7 @@ contract SafeERC7579 is
             uint48 validUntil,
             bytes calldata signatures
         ) = _getSafeOp(userOp);
-        try ISafe(payable(userOp.getSender())).checkSignatures(
+        try ISafe(payable(msg.sender)).checkSignatures(
             keccak256(operationData), operationData, signatures
         ) {
             // The timestamps are validated by the entry point,
@@ -397,7 +397,7 @@ contract SafeERC7579 is
             // result of `abi.encode`-ing the individual fields.
             EncodedSafeOpStruct memory encodedSafeOp = EncodedSafeOpStruct({
                 typeHash: SAFE_OP_TYPEHASH,
-                safe: userOp.sender,
+                safe: msg.sender,
                 nonce: userOp.nonce,
                 initCodeHash: keccak256(userOp.initCode),
                 callDataHash: keccak256(userOp.callData),