Fix: remove ISafe interface in the function handler (#99)

5afe · Sep 11, 2023 · 80daa43 · 80daa43
1 parent 3a6a6a3
commit 80daa43
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/contracts/base/FunctionHandlerManager.sol b/contracts/base/FunctionHandlerManager.sol
@@ -78,6 +78,6 @@ abstract contract FunctionHandlerManager is RegistryManager {
         // With safe v1.x, msg.data contains 20 bytes of sender address. Read the sender address by loading last 20 bytes.
         // remove last 20 bytes from calldata and store it in `data`.
         // Keep first 4 bytes (i.e function signature) so that handler contract can infer function identifier.
-        return ISafeProtocolFunctionHandler(functionHandler).handle(ISafe(safe), sender, 0, msg.data[0:(msg.data.length - 20)]);
+        return ISafeProtocolFunctionHandler(functionHandler).handle(safe, sender, 0, msg.data[0:(msg.data.length - 20)]);
     }
 }
diff --git a/contracts/interfaces/Modules.sol b/contracts/interfaces/Modules.sol
@@ -20,7 +20,7 @@ interface ISafeProtocolFunctionHandler is IERC165 {
      * @param data Arbitrary length bytes
      * @return result Arbitrary length bytes containing result of the operation
      */
-    function handle(ISafe safe, address sender, uint256 value, bytes calldata data) external returns (bytes memory result);
+    function handle(address safe, address sender, uint256 value, bytes calldata data) external returns (bytes memory result);
 
     /**
      * @notice A function that returns information about the type of metadata provider and its location.