Movatterモバイル変換


[0]ホーム

URL:


Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings

Commit3c5ce51

Browse files
committed
Fix pg_dumpall to restore its ability to dump from ancient servers.
Fix breakage induced by commitsd8d3d2aand463f262: pg_dumpall has crashed whenattempting to dump from pre-8.1 servers since then, due to faultyconstruction of the query used for dumping roles from older servers.The query was erroneous as of the earlier commit, but it wasn't exposedunless you tried to use --binary-upgrade, which you presumably wouldn'twith a pre-8.1 server. However commit463f262 made it fail always.In HEAD, also fix additional breakage induced in the same query bycommit491c029, which evidently wasn'ttested against pre-8.1 servers either.The bug is only latent in 9.1 because463f262 hadn't landed yet, butit seems best to back-patch all branches containing the faulty query.Gilles Darold
1 parent7eab804 commit3c5ce51

File tree

1 file changed

+2
-2
lines changed

1 file changed

+2
-2
lines changed

‎src/bin/pg_dump/pg_dumpall.c

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -679,7 +679,7 @@ dumpRoles(PGconn *conn)
679679
"ORDER BY 2");
680680
else
681681
printfPQExpBuffer(buf,
682-
"SELECT 0, usename as rolname, "
682+
"SELECT 0 as oid, usename as rolname, "
683683
"usesuper as rolsuper, "
684684
"true as rolinherit, "
685685
"usesuper as rolcreaterole, "
@@ -692,7 +692,7 @@ dumpRoles(PGconn *conn)
692692
"null as rolcomment "
693693
"FROM pg_shadow "
694694
"UNION ALL "
695-
"SELECT 0, groname as rolname, "
695+
"SELECT 0 as oid, groname as rolname, "
696696
"false as rolsuper, "
697697
"true as rolinherit, "
698698
"false as rolcreaterole, "

0 commit comments

Comments
 (0)

[8]ページ先頭

©2009-2025 Movatter.jp