Location via proxy:   [ UP ]  
[Report a bug]   [Manage cookies]                
Skip to content

Commit be09ceb

Browse files
committed
Fix pg_dumpall to restore its ability to dump from ancient servers.
Fix breakage induced by commits d8d3d2a and 463f262: pg_dumpall has crashed when attempting to dump from pre-8.1 servers since then, due to faulty construction of the query used for dumping roles from older servers. The query was erroneous as of the earlier commit, but it wasn't exposed unless you tried to use --binary-upgrade, which you presumably wouldn't with a pre-8.1 server. However commit 463f262 made it fail always. In HEAD, also fix additional breakage induced in the same query by commit 491c029, which evidently wasn't tested against pre-8.1 servers either. The bug is only latent in 9.1 because 463f262 hadn't landed yet, but it seems best to back-patch all branches containing the faulty query. Gilles Darold
1 parent 89fd41b commit be09ceb

File tree

1 file changed

+5
-3
lines changed

1 file changed

+5
-3
lines changed

src/bin/pg_dump/pg_dumpall.c

+5-3
Original file line numberDiff line numberDiff line change
@@ -714,7 +714,7 @@ dumpRoles(PGconn *conn)
714714
"ORDER BY 2");
715715
else
716716
printfPQExpBuffer(buf,
717-
"SELECT 0, usename as rolname, "
717+
"SELECT 0 as oid, usename as rolname, "
718718
"usesuper as rolsuper, "
719719
"true as rolinherit, "
720720
"usesuper as rolcreaterole, "
@@ -724,11 +724,12 @@ dumpRoles(PGconn *conn)
724724
"passwd as rolpassword, "
725725
"valuntil as rolvaliduntil, "
726726
"false as rolreplication, "
727+
"false as rolbypassrls, "
727728
"null as rolcomment, "
728729
"usename = current_user AS is_current_user "
729730
"FROM pg_shadow "
730731
"UNION ALL "
731-
"SELECT 0, groname as rolname, "
732+
"SELECT 0 as oid, groname as rolname, "
732733
"false as rolsuper, "
733734
"true as rolinherit, "
734735
"false as rolcreaterole, "
@@ -739,7 +740,8 @@ dumpRoles(PGconn *conn)
739740
"null::abstime as rolvaliduntil, "
740741
"false as rolreplication, "
741742
"false as rolbypassrls, "
742-
"null as rolcomment, false "
743+
"null as rolcomment, "
744+
"false AS is_current_user "
743745
"FROM pg_group "
744746
"WHERE NOT EXISTS (SELECT 1 FROM pg_shadow "
745747
" WHERE usename = groname) "

0 commit comments

Comments
 (0)