Temporarily switch back 2 safeApproves to approve

Openzeppelin has a bug in the safeApprove, we will need to write our own OpenZeppelin/openzeppelin-contracts#2219 We still use safeApprove in cases where we are sure the subsequent transferFrom switches the approve balance back to 0.
BrokkrFinance · Sep 22, 2022 · 7f08fc0 · 7f08fc0
1 parent 292131f
commit 7f08fc0
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/contracts/strategies/traderjoe/TraderJoe.sol b/contracts/strategies/traderjoe/TraderJoe.sol
@@ -94,11 +94,11 @@ contract TraderJoe is UUPSUpgradeable, StrategyOwnablePausableBaseUpgradeable {
         );
         uint256 depositTokenDesired = amount - swapAmount;
 
-        strategyStorage.pairDepositToken.safeApprove(
+        strategyStorage.pairDepositToken.approve(
             address(strategyStorage.router),
             pairDepositTokenDesired
         );
-        depositToken.safeApprove(
+        depositToken.approve(
             address(strategyStorage.router),
             depositTokenDesired
         );