Skip to content

Commit 6c16831

Browse files
committed
fix: apply consistent mapping order _operatorAuth (OZ_N-05)
1 parent 0990e9b commit 6c16831

File tree

2 files changed

+3
-3
lines changed

2 files changed

+3
-3
lines changed

packages/horizon/contracts/staking/HorizonStaking.sol

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -959,7 +959,7 @@ contract HorizonStaking is HorizonStakingBase, IHorizonStakingMain {
959959
if (_verifier == SUBGRAPH_DATA_SERVICE_ADDRESS) {
960960
_legacyOperatorAuth[msg.sender][_operator] = _allowed;
961961
} else {
962-
_operatorAuth[msg.sender][_verifier][_operator] = _allowed;
962+
_operatorAuth[msg.sender][_operator][_verifier] = _allowed;
963963
}
964964
emit OperatorSet(msg.sender, _operator, _verifier, _allowed);
965965
}
@@ -976,7 +976,7 @@ contract HorizonStaking is HorizonStakingBase, IHorizonStakingMain {
976976
if (_verifier == SUBGRAPH_DATA_SERVICE_ADDRESS) {
977977
return _legacyOperatorAuth[_serviceProvider][_operator];
978978
} else {
979-
return _operatorAuth[_serviceProvider][_verifier][_operator];
979+
return _operatorAuth[_serviceProvider][_operator][_verifier];
980980
}
981981
}
982982
}

packages/horizon/contracts/staking/HorizonStakingStorage.sol

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -158,7 +158,7 @@ abstract contract HorizonStakingV1Storage {
158158

159159
/// @dev Operator allow list
160160
/// Used for all verifiers except the subgraph data service.
161-
mapping(address serviceProvider => mapping(address verifier => mapping(address operator => bool authorized)))
161+
mapping(address serviceProvider => mapping(address operator => mapping(address verifier => bool authorized)))
162162
internal _operatorAuth;
163163

164164
/// @dev Flag to enable or disable delegation slashing

0 commit comments

Comments
 (0)