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

getNaughtyOrNice would return nice for an unset address

Summary

function getNaughtyOrNiceOnce(address person) external view returns (Status) {
return s_theListCheckedOnce[person];
}

Vulnerability Details

the function returns the status of an already set address , but in cases of an address that hasnt been set it will return Nice , the default value of the status

Impact

it makes the contract vulnerable to hijackers

Tools Used

manual review

Recommendations

the default should be changes and checked for

Updates

Lead Judging Commences

inallhonesty Lead Judge over 1 year ago
Submission Judgement Published
Invalidated
Reason: Lack of quality

Support

FAQs

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