The SantasList::checkList
function is not protected with the modifier SantasList::onlySanta
and therefore Anyone can call that function to set their Status which should not be done in this case.
If this function is not protected with the SantasList::onlySanta
modifier then it can be called by an external account or contract to set their own status and can get a higher favourable value than others which is not fair and against the rules of the contract
Anyone can set their Own status irrespective of what they have to get or assigned to. So that everyone will try to get higher returns which is an unfair advantage.
Manual Analysis
foundry
Use SantasList::onlySanta
modifier to SantasList::checkList
function aswell.
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.
The contest is live. Earn rewards by submitting a finding.
This is your time to appeal against judgements on your submissions.
Appeals are being carefully reviewed by our judges.