Submission Details
Severity:
safeTransfer must be used instead for transfers
Summary
token transfers are not using safeTransfers
Vulnerability Details
there are many instances where transfer and transferFrom are used, whereas token may have different implementations of transfers that result in challenges with ensuring tokens sent correctly, return values, token loss, reverts etc
Impact
Medium: not using safeTransfer creates problems with error handling,token loss, and proper reverting of transactions
Tools Used
Manual Analysis
Recommendations
It is recommended to make use of SafeERC20 wrapper for all token transfers e.g using SafeERC20 for IERC20 etc by using OpenZeppelin SafeERC20 library
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.