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

Error message should give some information

Summary

The custom error function should return some string with message to give a better vision of what's happened.

Vulnerability Details

Error event should return a message.

Impact

Easier debugging process and more information for the users.

Tools Used

Manual codebase analysis.

Recommendations

- error PasswordStore__NotOwner();
+ error PasswordStore__NotOwner(string message);
+ modifier onlyOwner() {
+ if (msg.sender != s_owner) {
+ revert PasswordStore__NotOwner("Only the owner can perform this operation.");
+ }
+ _;
+ }
Updates

Lead Judging Commences

inallhonesty Lead Judge
almost 2 years ago
inallhonesty Lead Judge almost 2 years ago
Submission Judgement Published
Invalidated
Reason: Non-acceptable severity

Support

FAQs

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