Submission Details
Severity:
Attacker can steal users tokens and buy present for himself
Summary
Attacker can steal user tokens and use them to buy nfts for himself by passing user address as a parameter to the buypresent function. Since the function charges the passed address for the token, the user token will be deducted for the trade instead for the msg.sender
Vulnerability Details
Logic error
Impact
loss of tokens
Tools Used
foundry
Recommendations
buyPresent should burn msg.sender tokens instead of presentReciever
Updates
Lead Judging Commences
inallhonesty almost 2 years 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.