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

```SantasList::UNKNOWN``` status is missing

Summary

The SantasList contract currently lacks the UNKNOWN status, which contradicts the documentation in the readme file. As per the documentation, the valid Status values should include:

  • NICE

  • EXTRA_NICE

  • NAUGHTY

  • UNKNOWN

The absence of the UNKNOWN status poses an issue for both the checkList() and checkTwice() functions, preventing them from accurately storing all possible status values.

Vulnerability Details

@> enum Status {
NICE,
EXTRA_NICE,
NAUGHTY,
NOT_CHECKED_TWICE
}

Impact

Santa can't store all the status correctly during the call of the functions checkList() and checkTwice().

Tools Used

Manual review.

Recommendations

Add the UNKNOWN status.

enum Status {
NICE,
EXTRA_NICE,
NAUGHTY,
+ UNKNOWN
NOT_CHECKED_TWICE
}
Updates

Lead Judging Commences

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.