First Flight #18: T-Swap

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

high miscalculation in getInputAmountBasedOnOutput in TSwapPoo

Summary

  ((inputReserves * outputAmount) * 10000) /
        ((outputReserves - outputAmount) * 997);

it should be 1000 on top in this equation

Vulnerability Details

Stealing funds from users, giving them wrong price.

Impact

Such miscalculations can lead to steal from users and loose all the credibility for protocol.

Tools Used

slither, code review

Recommendations

((inputReserves * outputAmount) * 1000) /
((outputReserves - outputAmount) * 997);

Updates

Appeal created

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

Incorrect fee calculation in TSwapPool::getInputAmountBasedOnOutput causes protocol to take too many tokens from users, resulting in lost fees

Support

FAQs

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