sherlock-audit / 2024-02-jala-swap-judging

6 stars 4 forks source link

goluu - Deadshare is cause of Loss of User Funds #227

Closed sherlock-admin2 closed 7 months ago

sherlock-admin2 commented 7 months ago

goluu

medium

Deadshare is cause of Loss of User Funds

Summary

Deadshare is a cause of the Loss of User Funds

Vulnerability Detail

-In Jala Swap makes good practice to transfer MINIMUM_LIQUIDITY address(0), Though it prevents major loss also causes of vulnerability.

loss of user fund

Code Snippet

https://github.com/sherlock-audit/2024-02-jala-swap/blob/main/jalaswap-dex-contract/contracts/JalaPair.sol#L147

Tool used

Manual Review

Recommendation

acknowledgment

nevillehuang commented 7 months ago

Invalid, this is intended to prevent inflation attacks as noted in issue #112