Submission Details
Severity:
Missing events - just a recommendation
Summary
I'd recommend adding in events for the selectWinner() and withdrawFees() functions
Vulnerability Details
These functions contain logic and state changes that could warrant being tracked offchain. For example maybe the feeRecipient wants to listen for when a winner is selected so they can go ahead and withdraw their fees.
Impact
Low - just a convenience but does not lead to loss of funds
Tools Used
Manual inspection
Recommendations
we can incorporate two new events
event WinnerSelected(address indexed winner);
event FeesWithdrawn(uint64 indexed fees);
Updates
Lead Judging Commences
Hamiltonite about 2 years ago
Submission Judgement Published Assigned finding tags:
events are missing or not detailed enough
Rewards breakdown
nSLOC
143This 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.