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

Fees contract swaps tokens on Uniswap with no slippage protection

Summary

Fees contract swaps tokens on Uniswap with no slippage protection.

Vulnerability Details

ISwapRouter.ExactInputSingleParams memory params = ISwapRouter
.ExactInputSingleParams({
tokenIn: _profits,
tokenOut: WETH,
fee: 3000,
recipient: address(this),
deadline: block.timestamp,
amountIn: amount,
amountOutMinimum: 0,
sqrtPriceLimitX96: 0
});

Above tokens are swapped and always use 0 for min out meaning the swapped tokens will be sandwiched and stolen.

Impact

Swapped tokens can be sandwiched and stolen.

Tools Used

Manual Review

Recommendations

Specify slippage parameters for all swapped tokens.

Support

FAQs

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

Give us feedback!