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

Commit52ad5fc

Browse files
committed
Don't access catalogs to validate GUCs when not connected to a DB.
Vignesh found this bug in the check function fordefault_table_access_method's check hook, but that was just copiedfrom older GUCs. Investigation by Michael and me then found the bug infurther places.When not connected to a database (e.g. in a walsender connection), wecannot perform (most) GUC checks that need database access. Even whenonly shared tables are needed, unless they'renailed (c.f. RelationCacheInitializePhase2()), they cannot be accessedwithout pg_class etc. being present.Fix by extending the existing IsTransactionState() checks to alsocheck for MyDatabaseOid.Reported-By: Vignesh C, Michael Paquier, Andres FreundAuthor: Vignesh C, Andres FreundDiscussion:https://postgr.es/m/CALDaNm1KXK9gbZfY-p_peRFm_XrBh1OwQO1Kk6Gig0c0fVZ2uw%40mail.gmail.comBackpatch: 9.4-
1 parent1eb8a5e commit52ad5fc

File tree

2 files changed

+13
-9
lines changed

2 files changed

+13
-9
lines changed

‎src/backend/commands/tablespace.c

Lines changed: 8 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -1062,10 +1062,11 @@ bool
10621062
check_default_tablespace(char**newval,void**extra,GucSourcesource)
10631063
{
10641064
/*
1065-
* If we aren't inside a transaction, we cannot do database access so
1066-
* cannot verify the name. Must accept the value on faith.
1065+
* If we aren't inside a transaction, or connected to a database, we
1066+
* cannot do the catalog accesses necessary to verify the name. Must
1067+
* accept the value on faith.
10671068
*/
1068-
if (IsTransactionState())
1069+
if (IsTransactionState()&&MyDatabaseId!=InvalidOid)
10691070
{
10701071
if (**newval!='\0'&&
10711072
!OidIsValid(get_tablespace_oid(*newval, true)))
@@ -1173,11 +1174,12 @@ check_temp_tablespaces(char **newval, void **extra, GucSource source)
11731174
}
11741175

11751176
/*
1176-
* If we aren't inside a transaction, we cannot do database access so
1177-
* cannot verify the individual names. Must accept the list on faith.
1177+
* If we aren't inside a transaction, or connected to a database, we
1178+
* cannot do the catalog accesses necessary to verify the name. Must
1179+
* accept the value on faith.
11781180
* Fortunately, there's then also no need to pass the data to fd.c.
11791181
*/
1180-
if (IsTransactionState())
1182+
if (IsTransactionState()&&MyDatabaseId!=InvalidOid)
11811183
{
11821184
temp_tablespaces_extra*myextra;
11831185
Oid*tblSpcs;

‎src/backend/utils/cache/ts_cache.c

Lines changed: 5 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -38,6 +38,7 @@
3838
#include"catalog/pg_ts_parser.h"
3939
#include"catalog/pg_ts_template.h"
4040
#include"commands/defrem.h"
41+
#include"miscadmin.h"
4142
#include"tsearch/ts_cache.h"
4243
#include"utils/builtins.h"
4344
#include"utils/catcache.h"
@@ -587,10 +588,11 @@ bool
587588
check_TSCurrentConfig(char**newval,void**extra,GucSourcesource)
588589
{
589590
/*
590-
* If we aren't inside a transaction, we cannot do database access so
591-
* cannot verify the config name. Must accept it on faith.
591+
* If we aren't inside a transaction, or connected to a database, we
592+
* cannot do the catalog accesses necessary to verify the config name.
593+
* Must accept it on faith.
592594
*/
593-
if (IsTransactionState())
595+
if (IsTransactionState()&&MyDatabaseId!=InvalidOid)
594596
{
595597
OidcfgId;
596598
HeapTupletuple;

0 commit comments

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