Submission Details
Severity:
Unhandled ERC20 transfers return values
Summary
Unhandled return values on ERC20 transfers
Vulnerability Details
ERC20 implementations are not always consistent. Some implementations of transfer and transferFrom could return false on failure instead of reverting. It is safer to wrap such calls into require() statements to these failures.
Impact
All non-standard ERC20 implementations that do not use reverts on failure, including DAI among others.
Recommendations
Use OpenZeppelin SafeTransfer/SafeTransferFrom or implement your own.
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.