15,000 USDC
View results
Submission Details
Severity: medium
Valid

`transfer` and `transferFrom` do not always have return values

For most tokens, transfer will either return true or revert, so there's no need to check for a true return value. However, for tokens like USDT, these functions do not return anything, causing the following line in the depositCollateral function to fail despite correct execution:
bool success = IERC20(tokenCollateralAddress).transferFrom(msg.sender, address(this), amountCollateral);

Support

FAQs

Can't find an answer? Chat with us on Discord, Twitter or Linkedin.