Submission Details
Severity:
checkList callable by anyone, which means anyone can change the status of another person's s_theListCheckedOnce, ruining their Christmas if they don't understand smart contracts
Summary
checkList can be called by anyone, a griefer can change another person's s_theListCheckedOnce.
Vulnerability Details
checkList should only be callable by Santa
Impact
Low Risk as it can be adjusted back, but Santa will have to go through his list again to check.
Tools Used
Manual Review
Recommendations
Include onlySanta modifier in checkList function.
Updates
Lead Judging Commences
inallhonesty over 1 year ago
Submission Judgement Published 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.
Rewards breakdown
nSLOC
116This contest has a flat reward structure with the following payouts:
100 EXP
20 EXP
2 EXP
Live
The contest is live. Earn rewards by submitting a finding.
Appeals
This is your time to appeal against judgements on your submissions.
Appeals Review
Appeals are being carefully reviewed by our judges.