First Flight #18: T-Swap

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

Double import of IERC20 from different libraries

Summary

Double import of IERC20
one implementation from openzeppelin in TSwapPool
and one in src/PoolFactory.sol importing from forge-std/interfaces/IERC20.sol and at the same importing TSwapPool which already have IERC20 imported

Vulnerability Details

Mismatch in implementations may cause some undesired results.

Impact

Low in this case as all functions used have similar implementations in both libraries

Tools Used

Static analyze and slither

Recommendations

delete additional IERC20 import in /PoolFactory.sol

Updates

Appeal created

inallhonesty Lead Judge over 1 year ago
Submission Judgement Published
Invalidated
Reason: Non-acceptable severity

Support

FAQs

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