From 364da52a49a8b308c7e8c1ba81f7f6b1907a6dd5 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Nicol=C3=A1s=20Venturo?= Date: Wed, 22 Apr 2020 17:17:33 -0300 Subject: [PATCH] Add note about default admin role security (#2211) * Add note about default admin security * Update contracts/access/AccessControl.sol Co-Authored-By: Francisco Giordano Co-authored-by: Francisco Giordano --- contracts/access/AccessControl.sol | 4 ++++ 1 file changed, 4 insertions(+) diff --git a/contracts/access/AccessControl.sol b/contracts/access/AccessControl.sol index cd7ebd9ca..3456b5a8c 100644 --- a/contracts/access/AccessControl.sol +++ b/contracts/access/AccessControl.sol @@ -34,6 +34,10 @@ import "../GSN/Context.sol"; * that only accounts with this role will be able to grant or revoke other * roles. More complex role relationships can be created by using * {_setRoleAdmin}. + * + * WARNING: The `DEFAULT_ADMIN_ROLE` is also its own admin: it has permission to + * grant and revoke this role. Extra precautions should be taken to secure + * accounts that have been granted it. */ abstract contract AccessControl is Context { using EnumerableSet for EnumerableSet.AddressSet;