Liquidity.Party/tycho-execution
2
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

feat: Use openzepplin's sendValue instead of send for ETH transfers

Browse Source 
--- don't change below this line ---
ENG-4260 Took 7 minutes
This commit is contained in:
Diana Carvalho
2025-02-21 11:37:06 +00:00
parent 530c86dbf3
commit 0ba5d02268
1 changed files with 1 additions and 4 deletions
Download Patch File Download Diff File Expand all files Collapse all files

5
foundry/src/TychoRouter.sol
View File

@@ -15,7 +15,6 @@ import "./Dispatcher.sol";
import {LibSwap} from "../lib/LibSwap.sol"; import {LibSwap} from "../lib/LibSwap.sol";
import {IPoolManager} from "@uniswap/v4-core/src/interfaces/IPoolManager.sol"; import {IPoolManager} from "@uniswap/v4-core/src/interfaces/IPoolManager.sol";
error TychoRouter__WithdrawalFailed();
error TychoRouter__AddressZero(); error TychoRouter__AddressZero();
error TychoRouter__EmptySwaps(); error TychoRouter__EmptySwaps();
error TychoRouter__NegativeSlippage(uint256 amount, uint256 minAmount); error TychoRouter__NegativeSlippage(uint256 amount, uint256 minAmount);
@@ -408,9 +407,7 @@ contract TychoRouter is AccessControl, Dispatcher, Pausable, ReentrancyGuard {
uint256 amount = address(this).balance; uint256 amount = address(this).balance;
if (amount > 0) { if (amount > 0) {
emit Withdrawal(address(0), amount, receiver); emit Withdrawal(address(0), amount, receiver);
// slither-disable-next-line arbitrary-send-eth Address.sendValue(payable(receiver), amount);
bool success = payable(receiver).send(amount);
if (!success) revert TychoRouter__WithdrawalFailed();
} }
} }