First Flight #18: T-Swap

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

`TSwapPool::LiquidityAdded` Event has parameters out of order

##Summary When the liquidity added, event is emitted in the TSwapPool::_addLiquidityMintAndTransfer function. it logs the values in incorrect order, The poolTokensToDeposit values should go as a 3rd parameter and wethToDeposit should be 2nd place
``

Description Event emission sis incorrect, leading to off chain functions potentially malfunctioning

##Impact Event emission sis incorrect, leading to off chain functions potentially malfunctioning

Tools Manual Review

##Recommendations event should be use in correct order

Updates

Appeal created

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

`LiquidityAdded` event has parameters out of order

Support

FAQs

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