summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorDaniel Gustafsson <dgustafsson@postgresql.org>2023-04-26 14:24:13 +0200
committerDaniel Gustafsson <dgustafsson@postgresql.org>2023-04-26 14:24:13 +0200
commit7b7fa85130330128b404eddebd4f33c6739454b0 (patch)
tree495af78578952e0d3266a1772139bba3c90143fe
parent4def50eba99e0a7e9b4c09fa3e33209d6d6fb834 (diff)
downloadpostgresql-7b7fa85130330128b404eddebd4f33c6739454b0.tar.gz
Fix stop condition for dumping GRANT commands
Commit ce6b672e44 changed dumping GRANT commands to ensure that grantors already have an ADMIN OPTION on the role for which it is granting permissions. Looping over the grants per role has a stop condition on dumping the grant statements, but accidentally missed updating the variable for the conditional check. Author: Andreas Scherbaum <ads@pgug.de> Co-authored-by: Artur Zakirov <zaartur@gmail.com> Discussion: https://postgr.es/m/de44299d-cd31-b41f-2c2a-161fa5e586a5@pgug.de
-rw-r--r--src/bin/pg_dump/pg_dumpall.c1
1 files changed, 1 insertions, 0 deletions
diff --git a/src/bin/pg_dump/pg_dumpall.c b/src/bin/pg_dump/pg_dumpall.c
index 71a1319865..988a0cd473 100644
--- a/src/bin/pg_dump/pg_dumpall.c
+++ b/src/bin/pg_dump/pg_dumpall.c
@@ -1052,6 +1052,7 @@ dumpRoleMembership(PGconn *conn)
PQfinish(conn);
exit_nicely(1);
}
+ prev_remaining = remaining;
/* Make one pass over the grants for this role. */
for (i = start; i < end; ++i)