Submission Details
Severity:
raffleDuration should be immutable
Summary
The variable PuppyRaffle::raffleDuration should be immutable to consume less gas.
Vulnerability Details
PuppyRaffle::raffleDuration is set by the constructor and is no longer modified, you can save gas by setting it as immutable.
address[] public players;
- uint256 public raffleDuration;
+ uint256 public immutable raffleDuration;
uint256 public raffleStartTime;
address public previousWinner;
Impact
The impact is minimal, just more gas is consumed.
Tools Used
Manual review
Recommendations
Set the variable as immutable.
Rewards breakdown
nSLOC
143This 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.