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 6 months ago
Submission Judgement Published
Invalidated
Reason: Design choice

Support

FAQs

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