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

Commitd1da801

Browse files
committed
Fix DROP DATABASE for databases with many ACLs
Commitc66a7d7 modified DROP DATABASE so that if interrupted, thedatabase is known to be in an invalid state and can only be dropped.This is done by setting a flag using an in-place update, so that it'snot lost in case of rollback.For databases with many ACLs, this may however fail like this: ERROR: wrong tuple lengthThis happens because with many ACLs, the pg_database.datacl attributegets TOASTed. The dropdb() code reads the tuple from the syscache, whichmeans it's detoasted. But the in-place update expects the tuple lengthto match the on-disk tuple.Fixed by reading the tuple from the catalog directly, not from syscache.Report and fix by Ayush Tiwari. Backpatch to 12. The DROP DATABASE fixwas backpatched to 11, but 11 is EOL at this point.Reported-by: Ayush TiwariAuthor: Ayush TiwariReviewed-by: Tomas VondraBackpatch-through: 12Discussion:https://postgr.es/m/CAJTYsWWNkCt+-UnMhg=BiCD3Mh8c2JdHLofPxsW3m2dkDFw8RA@mail.gmail.com
1 parenta95f13c commitd1da801

File tree

1 file changed

+16
-1
lines changed

1 file changed

+16
-1
lines changed

‎src/backend/commands/dbcommands.c

Lines changed: 16 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1637,6 +1637,8 @@ dropdb(const char *dbname, bool missing_ok, bool force)
16371637
booldb_istemplate;
16381638
Relationpgdbrel;
16391639
HeapTupletup;
1640+
ScanKeyDatascankey;
1641+
SysScanDescscan;
16401642
Form_pg_databasedatform;
16411643
intnotherbackends;
16421644
intnpreparedxacts;
@@ -1774,7 +1776,18 @@ dropdb(const char *dbname, bool missing_ok, bool force)
17741776
*/
17751777
pgstat_drop_database(db_id);
17761778

1777-
tup=SearchSysCacheCopy1(DATABASEOID,ObjectIdGetDatum(db_id));
1779+
/*
1780+
* Update the database's pg_database tuple
1781+
*/
1782+
ScanKeyInit(&scankey,
1783+
Anum_pg_database_datname,
1784+
BTEqualStrategyNumber,F_NAMEEQ,
1785+
CStringGetDatum(dbname));
1786+
1787+
scan=systable_beginscan(pgdbrel,DatabaseNameIndexId, true,
1788+
NULL,1,&scankey);
1789+
1790+
tup=systable_getnext(scan);
17781791
if (!HeapTupleIsValid(tup))
17791792
elog(ERROR,"cache lookup failed for database %u",db_id);
17801793
datform= (Form_pg_database)GETSTRUCT(tup);
@@ -1800,6 +1813,8 @@ dropdb(const char *dbname, bool missing_ok, bool force)
18001813
*/
18011814
CatalogTupleDelete(pgdbrel,&tup->t_self);
18021815

1816+
systable_endscan(scan);
1817+
18031818
/*
18041819
* Drop db-specific replication slots.
18051820
*/

0 commit comments

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