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

`SantasList::checkList ` function missing onlySanta modifier

Summary

SantasList::checkList function missing onlySanta modifier allow anybody to get the first check they want

Vulnerability Details

self explanitory

Impact

it removes the concept of double check

Tools Used

reading the code

Recommendations

replace

- function checkList(address person, Status status) external {
+ function checkList(address person, Status status) external onlySanta{
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.