Beginner FriendlyFoundryNFT
100 EXP
View results
Submission Details
Severity: low
Invalid

Missing checks for `address(0)` when assigning values to address state variables

[L-4]: Missing checks for address(0) when assigning values to address state variables

Check for address(0) when assigning values to address state variables.

  • Found in src/ChoosingRam.sol

    ramNFT = RamNFT(_ramNFT);
  • Found in src/Dussehra.sol

    ramNFT = RamNFT(_ramNFT);
    choosingRamContract = ChoosingRam(_choosingRamContract);
  • Found in src/RamNFT.sol

    choosingRamContract = _choosingRamContract;

Recommended Mitigation: Add a zero checks. These differ per case but follow the structure:

+ if(<ADDR> != address(0)) {
+ revert <CONTRACT_NAME>__ZeroCheckFailed();
+ }

Where is the address state variable and where <CONTRACT_NAME> is the contract name.

Updates

Lead Judging Commences

bube Lead Judge about 1 year ago
Submission Judgement Published
Invalidated
Reason: Non-acceptable severity
Assigned finding tags:

Info/Gas/Invalid according to docs

Support

FAQs

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