Unchecked return value of transferFrom in function timeLockERC20

Handle shw Vulnerability details Impact In the function timeLockERC20 line 610, the return value of IERC20.transferFrom is unchecked. The return value could be false if the transferred token is not ERC20-compliant, indicating that the transfer fails. In that case, the variable timelockERC20Balanc...

Unhandled return value of transferFrom in timeLockERC20() could lead to fund loss for recipients

Handle 0xRajeev Vulnerability details Impact ERC20 implementations are not always consistent. Some implementations of transfer and transferFrom could return ‘false’ on failure instead of reverting. It is safer to wrap such calls into require statements or use safe wrapper functions implementing...

Wrong TimeLockERC20 event emitted

Handle cmichel Vulnerability details Vulnerability Details The Visor.timeLockERC721 function emits the TimeLockERC20 event but should emit TimeLockERC721 instead. Impact It allows tricking the backend into registering ERC20 token transfers that never happened which could lead to serious issues wh...