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

Wrong Value in Status Enum

Summary

There exists a mismatch in value of Status in the code when compared with the documentation. These mismatches can often lead to unforeseen errors.

Vulnerability Details

The documentation states that there are 4 values for status like listed below:
NICE, EXTRA_NICE, NAUGHTY, or UNKNOWN

The contract contains the following 4 values for status as listed below:

enum Status {
NICE,
EXTRA_NICE,
NAUGHTY,
NOT_CHECKED_TWICE
}

Impact

The impact of the vulnerability is very low but it is good practice to follow the documentation to avoid any kind of error.

Tools Used

Manual Review

Recommendations

Change the value of NOT_CHECKED_TWICE to UNKNOWN

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

Lead Judging Commences

inallhonesty Lead Judge over 1 year ago
Submission Judgement Published
Invalidated
Reason: Non-acceptable severity

Support

FAQs

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