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

Commitfb694d9

Browse files
committed
pg_upgrade: only allow template0 to be non-connectable
This patch causes pg_upgrade to error out during its check phase if:(1) template0 is marked connectableor(2) any other database is marked non-connectableThis is done because, in the first case, pg_upgrade would fail becausethe pg_dumpall --globals restore would fail, and in the second case, thedatabase would not be restored, leading to data loss.Report by Matt Landry (1), Stephen Frost (2)Backpatch through 9.0
1 parent12cc299 commitfb694d9

File tree

1 file changed

+54
-0
lines changed

1 file changed

+54
-0
lines changed

‎src/bin/pg_upgrade/check.c

Lines changed: 54 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -19,6 +19,7 @@ static void check_databases_are_compatible(void);
1919
staticvoidcheck_locale_and_encoding(DbInfo*olddb,DbInfo*newdb);
2020
staticboolequivalent_locale(intcategory,constchar*loca,constchar*locb);
2121
staticvoidcheck_is_install_user(ClusterInfo*cluster);
22+
staticvoidcheck_proper_datallowconn(ClusterInfo*cluster);
2223
staticvoidcheck_for_prepared_transactions(ClusterInfo*cluster);
2324
staticvoidcheck_for_isn_and_int8_passing_mismatch(ClusterInfo*cluster);
2425
staticvoidcheck_for_reg_data_type_usage(ClusterInfo*cluster);
@@ -93,6 +94,7 @@ check_and_dump_old_cluster(bool live_check)
9394
* Check for various failure cases
9495
*/
9596
check_is_install_user(&old_cluster);
97+
check_proper_datallowconn(&old_cluster);
9698
check_for_prepared_transactions(&old_cluster);
9799
check_for_reg_data_type_usage(&old_cluster);
98100
check_for_isn_and_int8_passing_mismatch(&old_cluster);
@@ -642,6 +644,58 @@ check_is_install_user(ClusterInfo *cluster)
642644
}
643645

644646

647+
staticvoid
648+
check_proper_datallowconn(ClusterInfo*cluster)
649+
{
650+
intdbnum;
651+
PGconn*conn_template1;
652+
PGresult*dbres;
653+
intntups;
654+
inti_datname;
655+
inti_datallowconn;
656+
657+
prep_status("Checking database connection settings");
658+
659+
conn_template1=connectToServer(cluster,"template1");
660+
661+
/* get database names */
662+
dbres=executeQueryOrDie(conn_template1,
663+
"SELECTdatname, datallowconn "
664+
"FROMpg_catalog.pg_database");
665+
666+
i_datname=PQfnumber(dbres,"datname");
667+
i_datallowconn=PQfnumber(dbres,"datallowconn");
668+
669+
ntups=PQntuples(dbres);
670+
for (dbnum=0;dbnum<ntups;dbnum++)
671+
{
672+
char*datname=PQgetvalue(dbres,dbnum,i_datname);
673+
char*datallowconn=PQgetvalue(dbres,dbnum,i_datallowconn);
674+
675+
if (strcmp(datname,"template0")==0)
676+
{
677+
/* avoid restore failure when pg_dumpall tries to create template0 */
678+
if (strcmp(datallowconn,"t")==0)
679+
pg_fatal("template0 must not allow connections, "
680+
"i.e. its pg_database.datallowconn must be false\n");
681+
}
682+
else
683+
{
684+
/* avoid datallowconn == false databases from being skipped on restore */
685+
if (strcmp(datallowconn,"f")==0)
686+
pg_fatal("All non-template0 databases must allow connections, "
687+
"i.e. their pg_database.datallowconn must be true\n");
688+
}
689+
}
690+
691+
PQclear(dbres);
692+
693+
PQfinish(conn_template1);
694+
695+
check_ok();
696+
}
697+
698+
645699
/*
646700
*check_for_prepared_transactions()
647701
*

0 commit comments

Comments
 (0)
[8]ページ先頭
©2009-2025 Movatter.jp