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

test_owner_can_set_password() doesn't check non-owner

Summary

This test only checks the case when owner sets the password.

Vulnerability Details

Test doesn't check any other address except the owner's one. It is equally important to check a non-owner scenario.

Impact

This incomplete test results in the function setPassword() to keep working and not generating an error.

Tools Used

Manual check.

Recommendations

Add a check with non-owner address, e.g. vm.startPrank(address(1)).

Updates

Lead Judging Commences

inallhonesty Lead Judge
almost 2 years ago
inallhonesty Lead Judge almost 2 years ago
Submission Judgement Published
Invalidated
Reason: Other

Support

FAQs

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