Submission Details
Severity:
Missing access control at PasswordStore::setPassword()
-
Function
setPassword()from PasswordStore smart contract doesn't have access control implemented -
The comment from developer says
only the ownercan set a new password, but in reality anyone can execute the function with any argument -
To mitigate this issue, just copy the access control code from getPassword() to setPassword() too
if ( msg.sender != s_owner ) {
revert PasswordStore__NotOwner();
}
Updates
Lead Judging Commences
inallhonesty
about 2 years ago
inallhonesty about 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.