fix: relax underlying asset check on OracleSwapFreezer constructor (#447

)
aave · Feb 20, 2025 · 1936caa · 1936caa
1 parent b2b1801
commit 1936caa
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 3 deletions.
diff --git a/src/contracts/facilitators/gsm/swapFreezer/OracleSwapFreezer.sol b/src/contracts/facilitators/gsm/swapFreezer/OracleSwapFreezer.sol
@@ -54,7 +54,7 @@ contract OracleSwapFreezer is AutomationCompatibleInterface {
     uint128 unfreezeUpperBound,
     bool allowUnfreeze
   ) {
-    require(gsm.UNDERLYING_ASSET() == underlyingAsset, 'UNDERLYING_ASSET_MISMATCH');
+    require(underlyingAsset != address(0), 'ZERO_ADDRESS_NOT_VALID');
     require(
       _validateBounds(
         freezeLowerBound,

diff --git a/src/test/TestGsmOracleSwapFreezer.t.sol b/src/test/TestGsmOracleSwapFreezer.t.sol
@@ -26,8 +26,8 @@ contract TestGsmOracleSwapFreezer is TestGhoBase {
     GHO_GSM.grantRole(GSM_SWAP_FREEZER_ROLE, address(swapFreezer));
   }
 
-  function testRevertConstructorInvalidUnderlying() public {
-    vm.expectRevert('UNDERLYING_ASSET_MISMATCH');
+  function testRevertConstructorInvalidZeroAddress() public {
+    vm.expectRevert('ZERO_ADDRESS_NOT_VALID');
     new OracleSwapFreezer(
       GHO_GSM,
       address(0),