Skip to content

Commit a42bcbc

Browse files
committed
fix: make nonReentrant the first modifier (C4 QA)
1 parent d89c7a7 commit a42bcbc

File tree

1 file changed

+2
-2
lines changed

1 file changed

+2
-2
lines changed

contracts/l2/gateway/L2GraphTokenGateway.sol

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -141,7 +141,7 @@ contract L2GraphTokenGateway is GraphTokenGateway, L2ArbitrumMessenger, Reentran
141141
uint256, // unused on L2
142142
uint256, // unused on L2
143143
bytes calldata _data
144-
) public payable override notPaused nonReentrant returns (bytes memory) {
144+
) public payable override nonReentrant notPaused returns (bytes memory) {
145145
require(_l1Token == l1GRT, "TOKEN_NOT_GRT");
146146
require(_amount != 0, "INVALID_ZERO_AMOUNT");
147147
require(msg.value == 0, "INVALID_NONZERO_VALUE");
@@ -229,7 +229,7 @@ contract L2GraphTokenGateway is GraphTokenGateway, L2ArbitrumMessenger, Reentran
229229
address _to,
230230
uint256 _amount,
231231
bytes calldata _data
232-
) external payable override notPaused onlyL1Counterpart nonReentrant {
232+
) external payable override nonReentrant notPaused onlyL1Counterpart {
233233
require(_l1Token == l1GRT, "TOKEN_NOT_GRT");
234234
require(msg.value == 0, "INVALID_NONZERO_VALUE");
235235

0 commit comments

Comments
 (0)