Christmas Dinner

First Flight #31
Beginner FriendlyFoundrySolidity
100 EXP
View results
Submission Details
Severity: low
Valid

Lack of security checks leading to anyone becoming a participant

Summary

Anyone can become a participant bypassing the deposit function. Not enough checks in place.

Vulnerability Details

The else block changeParticipationStatus() can be called by anyone regardless of them making a deposit or not as this "!participant[msg.sender]" would always be false.

Impact

High

Tools Used

manual review

Recommendations

Consider adding more security checks/ requirements in else block of the changeParticipationStatus() when a user is not a participant.

Updates

Lead Judging Commences

0xtimefliez Lead Judge 7 months ago
Submission Judgement Published
Validated
Assigned finding tags:

usage of change participation logic circumvents deposit

Support

FAQs

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