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

misleading NatSpec in line 33

Summary

misleading NatSpec

Vulnerability Details

getPassword is a view function and it doesn't take any params, but the NatSpec says it takes "newPassword" as a param

/*
* @notice This allows only the owner to retrieve the password.
* @param newPassword The new password to set.
*/
function getPassword() external view returns (string memory) {
if (msg.sender != s_owner) {
revert PasswordStore__NotOwner();
}
return s_password;
}

Impact

can be misleading or create confusion

Recommendations

line no 33 should be removed from the codebase

Updates

Lead Judging Commences

inallhonesty Lead Judge
over 1 year ago
inallhonesty Lead Judge over 1 year ago
Submission Judgement Published
Invalidated
Reason: Non-acceptable severity

Support

FAQs

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