Closed code423n4 closed 2 years ago
defsec
On the helper contract, It has been observed that to safeIncreaseAllowance and safeDecreaseAllowance are commented out. (https://github.com/OpenZeppelin/openzeppelin-contracts/pull/2268) safeApprove function is deprecated.
https://github.com/code-423n4/2022-01-openleverage/blob/main/openleverage-contracts/contracts/lib/TransferHelper.sol#L49
Code Review
Consider to enable functions and use safeIncreaseAllowance and safeDecreaseAllowance instead of safeApprove.
Duplicate to #87
Handle
defsec
Vulnerability details
Impact
On the helper contract, It has been observed that to safeIncreaseAllowance and safeDecreaseAllowance are commented out. (https://github.com/OpenZeppelin/openzeppelin-contracts/pull/2268) safeApprove function is deprecated.
Proof of Concept
https://github.com/code-423n4/2022-01-openleverage/blob/main/openleverage-contracts/contracts/lib/TransferHelper.sol#L49
Tools Used
Code Review
Recommended Mitigation Steps
Consider to enable functions and use safeIncreaseAllowance and safeDecreaseAllowance instead of safeApprove.