Raisebox Faucet

First Flight #50
Beginner FriendlySolidity
100 EXP
View results
Submission Details
Impact: low
Likelihood: low
Invalid

Unused blockTime Variable

Root + Impact

Description

  • The contract defines a state variable:

uint256 public blockTime = block.timestamp;

This variable is only initialized at deployment and never used or updated anywhere in the code.

It has no functional purpose, yet it occupies a storage slot and increases deployment gas usage.

Risk

Impact:

  • Performance Overhead (Low):

    Adds one unnecessary storage slot, increasing deployment cost by ~20,000 gas.

  • Code Maintainability (Low):

    Misleads readers into thinking the variable plays a role in the contract’s timing logic.

Recommended Mitigation

@@ -39,8 +37,6 @@ contract RaiseBoxFaucet is ERC20, Ownable {
uint256 public dailySepEthCap;
- uint256 public blockTime = block.timestamp;
-
Updates

Lead Judging Commences

inallhonesty Lead Judge 7 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.