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

Non-protected Setter allow the caller to change password of the contracts

Summary

Function PasswordStore.setPassword(string) (src/PasswordStore.sol#26-29) is a non-protected setter s_password is written

Vulnerability Details

Usually, all setters must be protected with access control

Impact

Allow the caller to change the password of the contracts

Tools Used

Slither

Recommendations

Add access control

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.