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

the address parameter on buyPresent can be any arbitrary address

Summary

the address parameter should be msg.sender as this function can be called by an attacker on users address without consent

Vulnerability Details

the address parameter accepts arbitrary values and this can be called on any address asides the caller

Impact

it can be used to burn someones token without consent

Tools Used

Manual review

Recommendations

address should be 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.