<!-- canonical: https://0xsimao.com/findings/codeup-transfer-failed-codeup-erc20 -->

# Unused `TransferFailed` error in `CodeupERC20`

Low/Info · Three Sigma · DeFi game · 23rd October, 2024

Finding I-5 of the CODEUP security review.

- Report: /reports/codeup
- Source: https://cdn.sanity.io/files/qoqld077/staging/d9e57830a75b5165fd56e8b88037de7986a165ec.pdf

---

### Description

The `TransferFailed` error in `CodeupERC20` is not used and can be removed.

### Recommendation

Remove this error.

---

Related findings:

- [Erc20Utils::safeTransferFrom() assumes the tokens are transferred to this, which may not be the case](https://0xsimao.com/findings/districtone-erc20-utils-transfer-transferred): DistrictOne
- [transferFrom in ERC20Extended will always emit an Approval event if the allowance changes](https://0xsimao.com/findings/m-0-transfer-erc20-approval-allowance): M^0 Minter Gateway
