diff options
| author | Robert Haas <rhaas@postgresql.org> | 2012-03-21 12:38:34 -0400 |
|---|---|---|
| committer | Robert Haas <rhaas@postgresql.org> | 2012-03-21 12:39:48 -0400 |
| commit | 16f42be840bc7a96fb008055632388954c3548ba (patch) | |
| tree | 725f22f18beb97344be69f08a6de7c85faf477f4 /src/backend/executor/execMain.c | |
| parent | 04e9dc6e01dd8acb1b477c1649172dfbda822581 (diff) | |
| download | postgresql-16f42be840bc7a96fb008055632388954c3548ba.tar.gz postgresql-16f42be840bc7a96fb008055632388954c3548ba.zip | |
Don't allow CREATE TABLE AS to put relations in pg_global.
This was never intended to be allowed, and is blocked for an ordinary
CREATE TABLE, but CREATE TABLE AS slipped through the cracks. This
commit won't do anything to fix existing cases where this has loophole
has been exploited, but it still seems prudent to lock it down going
forward.
Back-branch commit only, as this problem has been refactored away
on the master branch.
Andres Freund
Diffstat (limited to 'src/backend/executor/execMain.c')
| -rw-r--r-- | src/backend/executor/execMain.c | 7 |
1 files changed, 7 insertions, 0 deletions
diff --git a/src/backend/executor/execMain.c b/src/backend/executor/execMain.c index 621ad8ac940..8a43db7d6ab 100644 --- a/src/backend/executor/execMain.c +++ b/src/backend/executor/execMain.c @@ -43,6 +43,7 @@ #include "access/xact.h" #include "catalog/heap.h" #include "catalog/namespace.h" +#include "catalog/pg_tablespace.h" #include "catalog/toasting.h" #include "commands/tablespace.h" #include "commands/trigger.h" @@ -2452,6 +2453,12 @@ OpenIntoRel(QueryDesc *queryDesc) get_tablespace_name(tablespaceId)); } + /* In all cases disallow placing user relations in pg_global */ + if (tablespaceId == GLOBALTABLESPACE_OID) + ereport(ERROR, + (errcode(ERRCODE_INVALID_PARAMETER_VALUE), + errmsg("only shared relations can be placed in pg_global tablespace"))); + /* Parse and validate any reloptions */ reloptions = transformRelOptions((Datum) 0, into->options, |