Submission Details
Severity:
Can arbitrarily burn someone else's SantaTokens to buy presents
Summary
Can arbitrarily burn someone else's SantaTokens to buy presents
Vulnerability Details
There is no check in buyPresent() to verify if presentReceiver is the same as msg.sender. This can be exploited to burn someone else's SantaTokens and mint NFTs for oneself.
Impact
Tools Used
Recommendations
add below check
require(presentReceiver == msg.sender)
Updates
Lead Judging Commences
inallhonesty over 1 year ago
Submission Judgement Published Assigned finding tags:
buyPresent should use msg.sender
Current implementation allows a malicious actor to burn someone else's tokens as the burn function doesn't actually check for approvals.
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.