Submission Details
Severity:
Function visibility
Summary
The 'setPassword' function is not protected by any modifier
Vulnerability Details
The 'setPassword' function can be called by any other EOA/contract.
Impact
Everybody can set a new password.
Tools Used
Manual Review
Recommendations
Add the following line of code:
if (msg.sender != s_owner) {
revert PasswordStore__NotOwner();
}
The best way is to create a modifier and apply it to the function (instead of copy-pasting) this code multiple times
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.