Submission Details
Severity:
[G-02] Use amout variable in transfer WETH
Vulnerability Details
The result for swapRouter.exactInputSingle(params) is the amountOut. So, since we are always going to transfer the amountOut, it is not necessary to use IERC20(WETH).balanceOf(address(this)) since we would get the same value that is available.
Impact
Optimization
Tools Used
Manual code review
Recommendations
Update the code for:
- IERC20(WETH).transfer(staking, IERC20(WETH).balanceOf(address(this)));
+ IERC20(WETH).transfer(staking, amount));
Prize pool breakdown
Total prize
20,000 USDC
nSLOC
70628 USDC / LOC
18,000 USDC
2,000 USDC
Live
The contest is live. Earn rewards by submitting a finding.
Appeals
This is your time to appeal against judgements on your submissions.
Appeals Review
Appeals are being carefully reviewed by our judges.