From 4a20fa621557d754cb677af41aa72de5cd7a6ffb Mon Sep 17 00:00:00 2001 From: TAMARA LIPOWSKI Date: Wed, 9 Apr 2025 15:10:02 -0400 Subject: [PATCH] fix: Conscious slither silencing - This arbitrary sender should be same, since the sender is always set to be msg.sender from within the contract (no calldata can be sent specifying otherwise). This is just used to circumvent msg.sender being the pool/protocol during callbacks. --- foundry/src/executors/ExecutorTransferMethods.sol | 1 + 1 file changed, 1 insertion(+) diff --git a/foundry/src/executors/ExecutorTransferMethods.sol b/foundry/src/executors/ExecutorTransferMethods.sol index 6eb73a8..ed4a141 100644 --- a/foundry/src/executors/ExecutorTransferMethods.sol +++ b/foundry/src/executors/ExecutorTransferMethods.sol @@ -40,6 +40,7 @@ contract ExecutorTransferMethods { if (method == TransferMethod.TRANSFER) { tokenIn.safeTransfer(receiver, amount); } else if (method == TransferMethod.TRANSFERFROM) { + // slither-disable-next-line arbitrary-send-erc20 tokenIn.safeTransferFrom(sender, receiver, amount); } else if (method == TransferMethod.TRANSFERPERMIT2) { // Permit2.permit is already called from the TychoRouter