NFT Dealers

First Flight #58
Beginner FriendlyFoundry
100 EXP
View results
Submission Details
Impact: low
Likelihood: low
Invalid

Redundant Address(0) Validation Check in Function


Description

The if block is redundant since address(0) cannot be the msg.sender in any case

// @> if (msg.sender == address(0)) revert InvalidAddress(); // @audit LOW useless if

Risk

Likelihood:

  • Low

Impact:

  • Low

Proof of Concept

Recommended Mitigation

remove the if block checking for address(0) in mintNft()

- if (msg.sender == address(0)) revert InvalidAddress();
Updates

Lead Judging Commences

rubik0n Lead Judge 16 days ago
Submission Judgement Published
Invalidated
Reason: Non-acceptable severity
Assigned finding tags:

info

Support

FAQs

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

Give us feedback!