First Flight #18: T-Swap

Summary:

When the LiquidityAddedevent is emitted in the TSwapPool::_addLiquidityMintAndTransfunction, it logs values in an incorrect order. The poolTokensToDeposit value should go in the third parameter position, whereas the wethToDeposit value should go second.

Vulnerability Details:

Impact:

Event emission is incorrect, leading to off-chain functions potentially malfunctioning.

Tools Used

Recommendations:

- emit LiquidityAdded(msg.sender, poolTokensToDeposit, wethToDeposit);
+ emit LiquidityAdded(msg.sender, wethToDeposit, poolTokensToDeposit);