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

Issue of access control in PasswordStore.sol contract

Summary

No restriction on who can set the password which makes the whole contract useless, making unauthorized individuals able to change the password.

Impact

Contract owner has no control over the contract

Tools Used

Manual Review

Recommendations

Consider using the "require checks" to ensure that the owner is the msg.sender, or consider importing and implementing the Ownable modifier from OpenZeppelin which makes it easy to also transfer the ownership of the contract.

Updates

Lead Judging Commences

inallhonesty Lead Judge
over 1 year ago
inallhonesty Lead Judge over 1 year 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.