Snowman Merkle Airdrop

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

Unused storage variable `SnowmanAirdrop::s_claimers`

Root + Impact

Description

The SnowmanAirdrop contract declares a private storage variable s_claimers, intended to store an array of addresses. However, this variable is neither read nor written to anywhere in the contract. Unused storage variables occupy storage slots in the contract layout, leading to increased deployment costs and unnecessary bytecode size. Additionally, their presence can cause confusion during code review or audits, potentially indicating incomplete functionality or abandoned logic.

Risk

Likelihood: Low
Impact: Low

While this issue does not introduce a functiona or security vulnerability, it results in gas inefficiencies and reduce overall code clarity. This can negatively impact maintainability and increase the risk of misunderstanding the contract's intended functionality.

Proof of Concept

address[] private s_claimers; // array to store addresses of claimers

This variable is not referenced in any read, write, or emit operations throughout the contract.

Recommended Mitigation

Remove the unused storage variable to reduce deployment overhead and improve maintainability.
If the variable was intended for future use, consider documenting that explicitly, or omit it until it is actually needed in the implementation.

- address[] private s_claimers; // array to store addresses of claimers
Updates

Lead Judging Commences

yeahchibyke Lead Judge 3 months ago
Submission Judgement Published
Invalidated
Reason: Non-acceptable severity

Support

FAQs

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