Part 2

Zaros
PerpetualsDEXFoundrySolidity
70,000 USDC
View results
Submission Details
Severity: low
Invalid

Use slippage parameter specified for a pair

Summary

The BaseAdapter.slippageToleranceBps variable is used for all swapped pairs ignoring the volatility difference. This way the owner has to set such slippageToleranceBps that is satisfied to a more volatile pair, i.e. the biggest.

Vulnerability Details

BaseAdapter.sol

uint256 public slippageToleranceBps;//@audit specified for a router but not for a pair

Impact

Weak slippage protection

Tools used

Manual Review

Recommendations

Consider specifying the slippageToleranceBps for pairs.

Updates

Lead Judging Commences

inallhonesty Lead Judge 10 months ago
Submission Judgement Published
Invalidated
Reason: Design choice

Support

FAQs

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

Give us feedback!