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

Arbitrary burn in buyPresent

Summary

Santa token can be burnt from other user with present minted to attacker

Vulnerability Details

Arbitrary Santa token holder address can be passed as presentReceiver. Token will be burnt from the arbitrary holder and present is mined to the attacker

Impact

Attacker receives nft presents by burning Santa token from holders

Tools Used

Recommendations

Burn token only from msg.sender, i_santaToken.burn(msg.sender);

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.