Snowman Merkle Airdrop

First Flight #42
Beginner FriendlyFoundrySolidityNFT
100 EXP
View results
Submission Details
Impact: low
Likelihood: medium
Invalid

Unused Claimers Address Array (s_claimers) Causes Unnecessary Storage Waste

Root + Impact

Description

  • The contract declares an address[] private s_claimers array intended to store addresses of claimers.

  • However, the array is never updated (no addresses are ever pushed), rendering it unused and occupying unnecessary storage space.

address[] private s_claimers; // @> declared but never updated or used

Risk

Likelihood:

  • This will always occur as the array is declared but never updated.

  • It does not affect contract logic or user interaction directly.

Impact:

  • Unnecessary storage consumes gas and increases deployment cost.

  • Maintaining unused variables adds code complexity and potential confusion.

Recommended Mitigation

- address[] private s_claimers;
+ // Remove s_claimers if not used

or keep track of claimers:

function claimSnowman(address receiver, ...) external nonReentrant {
+ s_claimers.push(receiver);
...
}
Updates

Lead Judging Commences

yeahchibyke Lead Judge 5 days ago
Submission Judgement Published
Invalidated
Reason: Non-acceptable severity

Support

FAQs

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