First Flight #18: T-Swap

First Flight #18
Beginner FriendlyDeFiFoundry
100 EXP
View results
Submission Details
Severity: high
Valid

No slippage protection

Summary: the swapExactOutput function does not include slippage protection

Vulnerability Details : the swapExactOutput function does not include slippage protection. this function is similar to what is done in TSwapPool::swapExactinput where the function specifies a minOutputAmount. but swapExactOutput is not mentioning maxOutputAmount or someting like that

Imact: High

##tools : Foundry Fuzz, Manual review

Recommendations : use uint256 maxAmoutOutput

Updates

Lead Judging Commences

inallhonesty Lead Judge
about 1 year ago

Appeal created

inallhonesty Lead Judge about 1 year ago
Submission Judgement Published
Validated
Assigned finding tags:

Lack of slippage protection in `TSwapPool::swapExactOutput` causes users to potentially receive way fewer tokens

Support

FAQs

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