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

QA/LOW: No input validation, user can pass empty string accidentally.

Summary

User can think they've saved their encrypted/unencrypted password, delete their local copies, and never being able to retrieve their password again, because they accidentally entered an empty string for the newPassword parameter.

Vulnerability Details

n/a

Impact

n/a

Tools Used

VSC.

Recommendations

Should add a check in this function to ensure the new password and old password cannot be equal:

function setPassword(string memory newPassword) external {
++ if (newPassword == s_password) revert NewPassword_Equal_OldPassword();
s_password = newPassword;
emit SetNetPassword();
}
Updates

Lead Judging Commences

inallhonesty Lead Judge
almost 2 years ago
inallhonesty Lead Judge almost 2 years ago
Submission Judgement Published
Invalidated
Reason: Admin Input/call validation

Support

FAQs

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