Christmas Dinner

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

receive() function doesn't distinguish between new and existing participants

Summary

receive() function doesn't distinguish between new and existing participants.

Vulnerability Details

The same NewSignup event is emitted regardless of whether the user sending funds is a new or existing participant.

Impact

The wrong event is emmitted.

Tools Used

Text editor.

Recommendations

Check whether msg.sender is already in the participant mapping and emit a NewSignup event if they aren't.

Updates

Lead Judging Commences

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

receive does not update participation status

Support

FAQs

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