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

setPassword() can be called by anyone causing the password to be changed by anyone

Summary

setPassword() function visibility is set to external and there is no access control or a logic to revert the function if the caller of the function is not msg.sender. Any EOA or contract can change the password.

Vulnerability Details

Impact

Anyone can change the password

Tools Used

  • Manual audit

Recommendations

Adding revert can be a solution to prevent others to change the password with the setPassword() function similar to what has been done in getPassword() function.

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.