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

Wrong comments & mispellings

Summary

Misspelling in event name and false comment on getPassword() function

Vulnerability Details

  1. Misspelled event SetNetPassword instead of event SetNewPassword)

  2. False comment at getPassword() function, where it says that it has @param newPassword The new password to set. variable, which is not true. Also, "this allows only the owner" seems like a word "function" is missed between "this" and "allows".

Impact

Confusing comment and name.

Recommendations

  1. Rename event SetNetPassword to SetNewPassword

  2. Remove @param newPassword from comment section. Replace "this allows" with "this function allows".

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.