Beginner FriendlyFoundry
100 EXP
View results
Submission Details
Severity: high
Valid

setPassword can be done by anyone, unknowingly to allow others to set/change password

Summary

setPassword can be change by anyone and should only be allowed of owner of password

Vulnerability Details

there are no requirements to require owner to be the only able to set/change password

Impact

High - security flaw

Tools Used

Solidity Docs/Remix

Recommendations

change line 27 to require(msg.sender == s_owner);

Updates

Lead Judging Commences

inallhonesty Lead Judge
almost 2 years ago
inallhonesty Lead Judge almost 2 years ago
Submission Judgement Published
Validated
Assigned finding tags:

finding-lacking-access-control

Anyone can call `setPassword` and set a new password contrary to the intended purpose.

Support

FAQs

Can't find an answer? Chat with us on Discord, Twitter or Linkedin.