Submission Details
Impact:
Likelihood:
Unused State Variable s_claimers Wastes Gas
Root + Impact
Description
The s_claimers array is declared but never populated or used anywhere in the contract. This wastes gas on deployment and indicates incomplete implementation or dead code.
// src/SnowmanAirdrop.sol:42
address[] private s_claimers; // @> Declared but never used
Risk
Likelihood:
-
Present in every deployment
-
Guaranteed to waste gas
Impact:
-
Unnecessary deployment gas cost
-
Code clutter and confusion
-
May indicate incomplete feature
Proof of Concept
// The array is declared but:
// 1. Never pushed to in claimSnowman()
// 2. No getter function to read it
// 3. No other function references it
// Result: Dead code wasting gas
Recommended Mitigation
- address[] private s_claimers;
Or if tracking claimers is desired:
function claimSnowman(address receiver, bytes32[] calldata merkleProof, uint8 v, bytes32 r, bytes32 s)
external
nonReentrant
{
// ... existing checks ...
i_snow.safeTransferFrom(receiver, address(this), amount);
s_hasClaimedSnowman[receiver] = true;
+ s_claimers.push(receiver);
emit SnowmanClaimedSuccessfully(receiver, amount);
i_snowman.mintSnowman(receiver, amount);
}
+ function getClaimers() external view returns (address[] memory) {
+ return s_claimers;
+ }
+
+ function getClaimersCount() external view returns (uint256) {
+ return s_claimers.length;
+ }
Rewards breakdown
This contest has a flat reward structure with the following payouts:
Live
The contest is live. Earn rewards by submitting a finding.
Judging
Submissions are being reviewed by our AI judge. Results will be available in a few minutes.
View all submissionsRewards Distribution
The contest is complete and the rewards are being distributed.