Lack of SqrtPriceLimitX96 can lead to high price impact
Summary
By setting SqrtPriceLimitX96 to 0, the swap router is vulnerable to potential high price impact if the pool lacks the liquidity to adequately support the trade.
Vulnerability Details
No price limit checks opens swaps up to high price impact, potentially causing unnecessary losses for users.
Impact
No price limit checks opens swaps up to high price impact, potentially causing unnecessary losses for users.
Tools Used
Manual Review
Recommendations
Price Impact should be considered before swapping. Hardcoding a maximum allowable price impact (ie 2%) will prevent swaps from occuring which incur high price impact, losing user funds. alternatively allow users to specify their price impact and pass this value in as an argument.
Prize pool breakdown
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.