Submission Details
Severity:
setPassword() has no owner check
Summary
Anyone change the users password
Vulnerability Details
setPassword() has no check to revert if the caller is not the owner of the contract. So anyone calling this function can overwrite the password with a new one of their choice.
Impact
users password can be overwritten
Tools Used
foundry
Recommendations
add a check+revert if the caller is not the owner
ie:
if (msg.sender != s_owner) { revert PasswordStore__NotOwner() }
Updates
Lead Judging Commences
inallhonesty
almost 2 years ago
inallhonesty almost 2 years ago
Submission Judgement Published Assigned finding tags:
finding-lacking-access-control
Anyone can call `setPassword` and set a new password contrary to the intended purpose.
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.