Submission Details
Severity:
Use safeTransfer/From instead of transfer/From
Summary
Use safeTransfer/From instead of transfer/From
Vulnerability Details
Tokens not compliant with the ERC20 specification could return false from the transfer function call to indicate the transfer fails, while the calling contract would not notice the failure if the return value is not checked.
It has more than 40 instances, so I will just add one relevant GitHub link.
Tools Used
Manual review
Recommendations
Use the SafeERC20 library implementation from OpenZeppelin and call safeTransfer or safeTransferFrom when transferring ERC20 tokens.
Prize pool breakdown
Total prize
20,000 USDC
nSLOC
70628 USDC / LOC
18,000 USDC
2,000 USDC
Live
The contest is live. Earn rewards by submitting a finding.
Appeals
This is your time to appeal against judgements on your submissions.
Appeals Review
Appeals are being carefully reviewed by our judges.