20,000 USDC
View results
Submission Details
Severity: low

Check ERC20 transfers

ERC20 transferFrom & transfer functions return a boolean value indicative of the transfer's success. This value is being ignored and not checked. This may lead to inconsistent state in the protocol.

Impact

This issue is widespread in the codebase.

Tools Used

Slither

Mitigation

Ideally, use SafeERC20 & safeTransfer and safeTransferFrom. Kindly, refer to Openzeppelin docs:
Openzeppelin SafeERC20

Support

FAQs

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