The Standard

Summary

There is no guarantee that ITokenManager::getAcceptedTokens() will return ERC20s with the same approve behavior.

Vulnerability Details

The unchecked return value of the approve() method can potentially cause transaction failures to go unnoticed in your contract.

Some IERC20 token implementations utilize boolean return values to indicate transaction failures, instead of relying on the revert() function. If the return value of the approve() method isn't appropriately verified, transactions may seemingly proceed even when the necessary token approvals have not been appropriately executed.

Tools Used

VSCode

Recommendations

Add a check o value returned by the approve() function