20,000 USDC
View results
Submission Details
Severity: high
Valid

No Slippage Parameter

Summary

The contracts lack slippage checks, making them vulnerable to sandwich attacks.

Vulnerability Details

function sellProfits(address _profits) public {
require(_profits != WETH, "not allowed");
uint256 amount = IERC20(_profits).balanceOf(address(this));
ISwapRouter.ExactInputSingleParams memory params = ISwapRouter
.ExactInputSingleParams({
tokenIn: _profits,
tokenOut: WETH,
fee: 3000,
recipient: address(this),
deadline: block.timestamp,
amountIn: amount,
amountOutMinimum: 0, //@audit no slippage check
sqrtPriceLimitX96: 0
});
amount = swapRouter.exactInputSingle(params);
IERC20(WETH).transfer(staking, IERC20(WETH).balanceOf(address(this)));
}

Impact

Vulnerable to sandwich attacks and MEV.

Tools Used

None

Recommendations

Implement checks for minimum return amounts.

Support

FAQs

Can't find an answer? Chat with us on Discord, Twitter or Linkedin.