Summary

Consider using safeApprove()

Vulnerability Details

• For some tokens, you should reset the allowance to zero before any approve.

• For some tokens there are a bool return value in approve.

By replacing the approve() with safeApprove(), you won't be at risk of these issues.

Impact

Some approves may revert or return false.

Tools Used

Manual Review

Recommendations

Consider using safeApprove()