Beginner FriendlyFoundry
100 EXP
View results
Submission Details
Severity: high
Valid

Attacker can burn others SantaToken via buyPresent function

Summary

Attack can call buyPresent function with others address to burn others SantaToken.

Vulnerability Details

SantaToken is burn from presentReceiver address not msg.sender, so attack can pass in any address to burn token from.

POC

vm.startPrank(attacker);
santasList.buyPresent(victim);
vm.stopPrank();

Impact

Attack can to burn others' SantaToken without limit.

Tools Used

Foundry

Recommendations

burn SantaToken from msg.sender, not presentReceiver

Updates

Lead Judging Commences

inallhonesty Lead Judge over 1 year ago
Submission Judgement Published
Validated
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.

Support

FAQs

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