Submission Details
Severity:
s_owner not being set to immutable, therefore wastes gas
Summary
The PasswordStore::s_owner variable is unable to be changed, therefore it should be immutable to save gas.
Vulnerability Details
contract PasswordStore {
error PasswordStore__NotOwner();
@> address private s_owner;
string private s_password;
If a variable never changes, then reading from storage is less gas efficient, and instead we should read from the bytecode. Using an immutable variable will enable the protocol to save gas on reading the PasswordStore::s_owner variable.
Impact
Wastes gas
Tools Used
Slither
Foundry
Recommendations
Set s_owner to immutable.
- address private s_owner;
+ address private immutable i_owner;
Updates
Rewards breakdown
nSLOC
20This 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.