NFT Dealers

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

Redundant Address(0) Validation Check in Function

Author Revealed upon completion

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();

Support

FAQs

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

Give us feedback!