Skip to content

Commit 3a6c3b5

Browse files
committed
fix: check before adding/removing whitelisted addresses [N-23]
1 parent 960cc04 commit 3a6c3b5

File tree

1 file changed

+2
-0
lines changed

1 file changed

+2
-0
lines changed

contracts/gateway/L1GraphTokenGateway.sol

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -149,6 +149,7 @@ contract L1GraphTokenGateway is GraphTokenGateway, L1ArbitrumMessenger {
149149
*/
150150
function addToCallhookWhitelist(address _newWhitelisted) external onlyGovernor {
151151
require(_newWhitelisted != address(0), "INVALID_ADDRESS");
152+
require(!callhookWhitelist[_newWhitelisted], "ALREADY_WHITELISTED");
152153
callhookWhitelist[_newWhitelisted] = true;
153154
emit AddedToCallhookWhitelist(_newWhitelisted);
154155
}
@@ -160,6 +161,7 @@ contract L1GraphTokenGateway is GraphTokenGateway, L1ArbitrumMessenger {
160161
*/
161162
function removeFromCallhookWhitelist(address _notWhitelisted) external onlyGovernor {
162163
require(_notWhitelisted != address(0), "INVALID_ADDRESS");
164+
require(callhookWhitelist[_notWhitelisted], "NOT_WHITELISTED");
163165
callhookWhitelist[_notWhitelisted] = false;
164166
emit RemovedFromCallhookWhitelist(_notWhitelisted);
165167
}

0 commit comments

Comments
 (0)