Skip to content

Commit e0440a8

Browse files
committed
fix(c4-6-g08): duplicated require()/revert() checks should be refactored to a modifier or
function Signed-off-by: Tomás Migone <[email protected]>
1 parent 2ad9f4e commit e0440a8

File tree

1 file changed

+2
-2
lines changed

1 file changed

+2
-2
lines changed

contracts/l2/token/GraphTokenUpgradeable.sol

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -114,15 +114,15 @@ contract GraphTokenUpgradeable is GraphUpgradeable, Governed, ERC20BurnableUpgra
114114
* @param _account Address of the minter
115115
*/
116116
function removeMinter(address _account) external onlyGovernor {
117-
require(_minters[_account], "NOT_A_MINTER");
117+
require(isMinter(_account), "NOT_A_MINTER");
118118
_removeMinter(_account);
119119
}
120120

121121
/**
122122
* @dev Renounce to be a minter.
123123
*/
124124
function renounceMinter() external {
125-
require(_minters[msg.sender], "NOT_A_MINTER");
125+
require(isMinter(msg.sender), "NOT_A_MINTER");
126126
_removeMinter(msg.sender);
127127
}
128128

0 commit comments

Comments
 (0)