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);
The contest is live. Earn rewards by submitting a finding.
This is your time to appeal against judgements on your submissions.
Appeals are being carefully reviewed by our judges.