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

Missing onlySanta modifier in `SantasList::checkList` allows anyone to call

Summary

SantasList::checkList should only be callable by Santa but is missing the onlySanta modifier.

Vulnerability Details

See summary

Impact

Anyone can set any status they choose on the first pass

Tools Used

Manual review

Recommendations

Add the missing onlySanta modifier to SantasList::checkList

Updates

Lead Judging Commences

inallhonesty Lead Judge over 1 year ago
Submission Judgement Published
Validated
Assigned finding tags:

Access Control on checkList()

Anyone is able to call checkList() changing the status of a provided address. This is not intended functionality and is meant to be callable by only Santa.

Support

FAQs

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