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

Lack of access control on checkList()

Summary

Lack of access control on the checkList() function of SantaList.sol allows anyone to check their status in the contract this will allow user to mint NFTS without being NICE OR EXTRA_NICE

Vulnerability Details

ALICE: call this function with her address as parameter and status NICE
ALICE: Can call collectPresent() afterward so that she can get an NFT, she can do this unlimited times with new wallets.

Impact

High

Tools Used

Manual Analysis

Recommendations

Add the onlySanta modifier to the function in order to be called only by Santa

Updates

Lead Judging Commences

inallhonesty Lead Judge almost 2 years 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.