Pieces Protocol

First Flight #32
Beginner FriendlyFoundrySolidityNFT
100 EXP
View results
Submission Details
Severity: medium
Invalid

No check for transfer()/transferFrom() return value

Summary

The usage of transfer() and transferFrom() requires checking the return value to verify the validation of the transaction overall

Vulnerability Details

Contract does not check for the return value of a ERC20 token transfer, which can cause unexpected behavior.

Impact

Incorrect ERC20 token balances or unexpected behavior.

Tools Used

Manual Inspection.

Recommended Mitigation

Check if the return value of transfer*() was false, if so revert() .

Updates

Lead Judging Commences

fishy Lead Judge 4 months ago
Submission Judgement Published
Invalidated
Reason: Incorrect statement

Support

FAQs

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