diff --git a/doc/src/sgml/ref/alter_role.sgml b/doc/src/sgml/ref/alter_role.sgml
index 7b368b53770..d2d437c7258 100644
--- a/doc/src/sgml/ref/alter_role.sgml
+++ b/doc/src/sgml/ref/alter_role.sgml
@@ -70,8 +70,10 @@ ALTER ROLE { role_specification | A
for that.)
Attributes not mentioned in the command retain their previous settings.
Database superusers can change any of these settings for any role.
- Roles having CREATEROLE> privilege can change any of these
- settings, but only for non-superuser and non-replication roles.
+ Roles having CREATEROLE privilege can change any of these
+ settings except SUPERUSER, REPLICATION,
+ and BYPASSRLS; but only for non-superuser and
+ non-replication roles.
Ordinary roles can only change their own password.
diff --git a/doc/src/sgml/ref/create_role.sgml b/doc/src/sgml/ref/create_role.sgml
index c23cffa53e7..28ba2cbb2b2 100644
--- a/doc/src/sgml/ref/create_role.sgml
+++ b/doc/src/sgml/ref/create_role.sgml
@@ -187,6 +187,8 @@ CREATE ROLE name [ [ WITH ] NOREPLICATION is the default.
+ You must be a superuser to create a new role having the
+ REPLICATION attribute.
@@ -198,11 +200,16 @@ CREATE ROLE name [ [ WITH ]
These clauses determine whether a role bypasses every row-level
security (RLS) policy. NOBYPASSRLS is the default.
+ You must be a superuser to create a new role having
+ the BYPASSRLS attribute.
+
+
+
Note that pg_dump will set row_security to
OFF by default, to ensure all contents of a table are
dumped out. If the user running pg_dump does not have appropriate
- permissions, an error will be returned. The superuser and owner of the
- table being dumped always bypass RLS.
+ permissions, an error will be returned. However, superusers and the
+ owner of the table being dumped always bypass RLS.
diff --git a/src/backend/commands/user.c b/src/backend/commands/user.c
index ccacf2d0d1f..7567e2f17e7 100644
--- a/src/backend/commands/user.c
+++ b/src/backend/commands/user.c
@@ -665,8 +665,10 @@ AlterRole(AlterRoleStmt *stmt)
roleid = HeapTupleGetOid(tuple);
/*
- * To mess with a superuser you gotta be superuser; else you need
- * createrole, or just want to change your own password
+ * To mess with a superuser or replication role in any way you gotta be
+ * superuser. We also insist on superuser to change the BYPASSRLS
+ * property. Otherwise, if you don't have createrole, you're only allowed
+ * to change your own password.
*/
if (authform->rolsuper || issuper >= 0)
{
@@ -682,7 +684,7 @@ AlterRole(AlterRoleStmt *stmt)
(errcode(ERRCODE_INSUFFICIENT_PRIVILEGE),
errmsg("must be superuser to alter replication users")));
}
- else if (authform->rolbypassrls || bypassrls >= 0)
+ else if (bypassrls >= 0)
{
if (!superuser())
ereport(ERROR,
@@ -691,11 +693,11 @@ AlterRole(AlterRoleStmt *stmt)
}
else if (!have_createrole_privilege())
{
+ /* We already checked issuper, isreplication, and bypassrls */
if (!(inherit < 0 &&
createrole < 0 &&
createdb < 0 &&
canlogin < 0 &&
- isreplication < 0 &&
!dconnlimit &&
!rolemembers &&
!validUntil &&