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

Commitec3e183

Browse files
committed
Disallow deletion of CurrentExtensionObject while running extension script.
While the deletion in itself wouldn't break things, any further creationof objects in the script would result in dangling pg_depend entries beingadded by recordDependencyOnCurrentExtension(). An example from PhilSorber convinced me that this is just barely likely enough to be worthexpending a couple lines of code to defend against. The resulting errormessage might be confusing, but it's better than leaving corrupted catalogcontents for the user to deal with.
1 parent8ab9df0 commitec3e183

File tree

1 file changed

+17
-0
lines changed

1 file changed

+17
-0
lines changed

‎src/backend/commands/extension.c‎

Lines changed: 17 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1642,6 +1642,23 @@ RemoveExtensionById(Oid extId)
16421642
HeapTupletuple;
16431643
ScanKeyDataentry[1];
16441644

1645+
/*
1646+
* Disallow deletion of any extension that's currently open for insertion;
1647+
* else subsequent executions of recordDependencyOnCurrentExtension()
1648+
* could create dangling pg_depend records that refer to a no-longer-valid
1649+
* pg_extension OID. This is needed not so much because we think people
1650+
* might write "DROP EXTENSION foo" in foo's own script files, as because
1651+
* errors in dependency management in extension script files could give
1652+
* rise to cases where an extension is dropped as a result of recursing
1653+
* from some contained object. Because of that, we must test for the case
1654+
* here, not at some higher level of the DROP EXTENSION command.
1655+
*/
1656+
if (extId==CurrentExtensionObject)
1657+
ereport(ERROR,
1658+
(errcode(ERRCODE_OBJECT_NOT_IN_PREREQUISITE_STATE),
1659+
errmsg("cannot drop extension \"%s\" because it is being modified",
1660+
get_extension_name(extId))));
1661+
16451662
rel=heap_open(ExtensionRelationId,RowExclusiveLock);
16461663

16471664
ScanKeyInit(&entry[0],

0 commit comments

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