First Flight #18: T-Swap

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

Lacking zero address checks

Description: Zero address checking is mandatory because the owner of the contract or token will lose the ownership of the contract.

Impact: Its' impact will be low because it will just transfer the ownership of the token and the contract.

Recommended Mitigation:

constructor(address wethToken) {
+ if(wethToken == address(0)) {
+ revert();
+ }
i_wethToken = wethToken;
}

Also, check that the address is not zero.

Updates

Appeal created

inallhonesty Lead Judge about 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.