Tadle

Summary

TokenManager.sol tillIn and withdraw functions do not revert on 0 amount but return, which could induce errors in another contract using them if it's not expecting a successful function execution but a revert

Vulnerability Details

TokenManager.sol tillIn and withdraw functions do not revert on 0 amount but return, which could induce errors in another contract using them if it's not expecting a successful function execution but a revert.

Impact

Not properly handling the successful function call could result in a wrong state or a wrong execution of other functions that use the TokenManager's tillInand withdrawfunctions.

Tools Used

Manual review

Recommendations

Change the empty returns on lines #69 and #146 to revert Errors.AmountIsZero();