First Flight #18: T-Swap

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

wethToken lack zero address check

Summary

The factory need to check if the provided weth token address is not zero

Vulnerability Details

In PoolFactory.constructor

constructor(address wethToken) {
i_wethToken = wethToken;
}

Impact

A human mistake can cause the factory to create invalid pools that will lock user's tokens.

Tools Used

Slither

Recommendations

constructor(address wethToken) {
require(wethToken != address(0));
i_wethToken = wethToken;
}
Updates

Appeal created

inallhonesty Lead Judge 12 months ago
Submission Judgement Published
Invalidated
Reason: Non-acceptable severity

Support

FAQs

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