fix: apply consistent mapping order inside ProvisionManager.sol (OZ_N…

…-05)
graphprotocol · Oct 3, 2024 · bf97048 · bf97048
1 parent 63c373e
commit bf97048
Showing 1 changed file with 3 additions and 3 deletions.
diff --git a/packages/horizon/contracts/data-service/utilities/ProvisionManager.sol b/packages/horizon/contracts/data-service/utilities/ProvisionManager.sol
@@ -71,7 +71,7 @@ abstract contract ProvisionManager is Initializable, GraphDirectory, ProvisionMa
      * @param caller The address of the caller.
      * @param serviceProvider The address of the service provider.
      */
-    error ProvisionManagerNotAuthorized(address caller, address serviceProvider);
+    error ProvisionManagerNotAuthorized(address serviceProvider, address caller);
 
     /**
      * @notice Thrown when a provision is not found.
@@ -84,8 +84,8 @@ abstract contract ProvisionManager is Initializable, GraphDirectory, ProvisionMa
      */
     modifier onlyProvisionAuthorized(address serviceProvider) {
         require(
-            _graphStaking().isAuthorized(msg.sender, serviceProvider, address(this)),
-            ProvisionManagerNotAuthorized(msg.sender, serviceProvider)
+            _graphStaking().isAuthorized(serviceProvider, msg.sender, address(this)),
+            ProvisionManagerNotAuthorized(serviceProvider, msg.sender)
         );
         _;
     }