Submission Details
Severity:
Missing emitted events for important state changes
Summary
Missing events emissions for important state changes
Vulnerability Details
Both SantasList::collectPresent and SantasList::buyPresent are the 2 of the most important functions in the protocol and fail to emit an event to notify the off chain resources.
Impact
LOW as no contract logic is affected
Tools Used
Manual Review
Recommendations
Add missing events to the contract
event PresentCollected(address user, uint256 tokenId);
event PresentBought(address user, address receiver, uint256 tokenId);
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.