Submission Details
Impact:
Likelihood:
Unused state variable increases gas costs
Author Revealed upon completion
Unused state variable increases gas costs
Description
src/SnowmanAirdrop.sol contains an unused state variable s_claimers
Unused State Variables are stored on-chain, and storing data on Ethereum is expensive.
// Root cause in the codebase with @> marks to highlight the relevant section
// state variables is unused in contract
@> address[] private s_claimers; // array to store addresses of claimers
Risk
In Future developers or builders might get confused about the purpose of a variable, leading to accidental misuse or introduction of bugs
Proof of Concept
// unused state variable throughout contract
@> address[] private s_claimers; // array to store addresses of claimers
Recommended Mitigation
Either we can use the variable in SnowmanAirdrop::claimSnowman(), when claimer claims the snowman NFT we can store the address of claimers.
- remove this code
@> address[] private s_claimers; // array to store addresses of claimers
Rewards breakdown
nSLOC
215This contest has a flat reward structure with the following payouts:
100 EXP
20 EXP
2 EXP
Live
The contest is live. Earn rewards by submitting a finding.
Appeals
This is your time to appeal against judgements on your submissions.
Appeals Review
Appeals are being carefully reviewed by our judges.
Rewards Distribution
The contest is complete and the rewards are being distributed.