From 0ba5d02268e42e10ab9a7d4390eb87dfd7099f07 Mon Sep 17 00:00:00 2001 From: Diana Carvalho Date: Fri, 21 Feb 2025 11:37:06 +0000 Subject: [PATCH] feat: Use openzepplin's sendValue instead of send for ETH transfers --- don't change below this line --- ENG-4260 Took 7 minutes --- foundry/src/TychoRouter.sol | 5 +---- 1 file changed, 1 insertion(+), 4 deletions(-) diff --git a/foundry/src/TychoRouter.sol b/foundry/src/TychoRouter.sol index 41ddde3..b009233 100644 --- a/foundry/src/TychoRouter.sol +++ b/foundry/src/TychoRouter.sol @@ -15,7 +15,6 @@ import "./Dispatcher.sol"; import {LibSwap} from "../lib/LibSwap.sol"; import {IPoolManager} from "@uniswap/v4-core/src/interfaces/IPoolManager.sol"; -error TychoRouter__WithdrawalFailed(); error TychoRouter__AddressZero(); error TychoRouter__EmptySwaps(); error TychoRouter__NegativeSlippage(uint256 amount, uint256 minAmount); @@ -408,9 +407,7 @@ contract TychoRouter is AccessControl, Dispatcher, Pausable, ReentrancyGuard { uint256 amount = address(this).balance; if (amount > 0) { emit Withdrawal(address(0), amount, receiver); - // slither-disable-next-line arbitrary-send-eth - bool success = payable(receiver).send(amount); - if (!success) revert TychoRouter__WithdrawalFailed(); + Address.sendValue(payable(receiver), amount); } }