Submission Details
Severity:
`Lender.sol` should use `safeTransfer` and `safeTransferFrom`
Summary
Some ERC20 token does not return true/false such USDT. So Lender.sol should use safeTransfer and safeTransferFrom. It may cause transfer error but user dose not know.
Vulnerability Details
https://github.com/Cyfrin/2023-07-beedle/blob/main/src/Lender.sol#L152
Impact
Some ERC20 token does not return true/false such USDT. So Lender.sol should use safeTransfer and safeTransferFrom. It may cause transfer error but user dose not know.
Tools Used
vs code
Recommendations
please check Lender.sol, and replace all the IERC20.transferFrom and IERC20.transfer
Lender.sol should use safeTransfer and safeTransferFrom
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.