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

Access control

Summary

Add the require statement or the if-revert statement in the setPassword function. Because only the owner can set the password.

Vulnerability Details

if(msg.sender != s_owner) {
revert PasswordStore_NotOwner();
}

Impact

If you not written require statement. Anybody can set the password.

Tools Used

Mannual

Recommendations

Add Checks. Follow checks, effects, interactions.

Updates

Lead Judging Commences

inallhonesty Lead Judge
about 2 years ago
inallhonesty Lead Judge about 2 years ago
Submission Judgement Published
Invalidated
Reason: Vague generalities

Support

FAQs

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