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

Commit8810356

Browse files
committed
Disallow setting bogus GUCs within an extension's reserved namespace.
Commit75d2206 tried to throw a warning for setting a custom GUC whoseprefix belongs to a previously-loaded extension, if there is no such GUCdefined by the extension. But that caused unstable behavior withparallel workers, because workers don't necessarily load extensions andGUCs in the same order their leader did. To make that work safely, wehave to completely disallow the case. We now actually remove any suchGUCs at the time of initial extension load, and then throw an error notjust a warning if you try to add one later. While this might create acompatibility issue for a few people, the improvement in error-detectioncapability seems worth it; it's hard to believe that there's any gooduse-case for choosing such GUC names.This also un-reverts5609cc0 (Rename EmitWarningsOnPlaceholders() toMarkGUCPrefixReserved()), since that function's old name is now evenmore of a misnomer.Florin Irion and Tom LaneDiscussion:https://postgr.es/m/1902182.1640711215@sss.pgh.pa.us
1 parent2776922 commit8810356

File tree

18 files changed

+101
-31
lines changed

18 files changed

+101
-31
lines changed

‎contrib/auth_delay/auth_delay.c‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -68,7 +68,7 @@ _PG_init(void)
6868
NULL,
6969
NULL);
7070

71-
EmitWarningsOnPlaceholders("auth_delay");
71+
MarkGUCPrefixReserved("auth_delay");
7272

7373
/* Install Hooks */
7474
original_client_auth_hook=ClientAuthentication_hook;

‎contrib/auto_explain/auto_explain.c‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -231,7 +231,7 @@ _PG_init(void)
231231
NULL,
232232
NULL);
233233

234-
EmitWarningsOnPlaceholders("auto_explain");
234+
MarkGUCPrefixReserved("auto_explain");
235235

236236
/* Install hooks. */
237237
prev_ExecutorStart=ExecutorStart_hook;

‎contrib/basic_archive/basic_archive.c‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -69,7 +69,7 @@ _PG_init(void)
6969
0,
7070
check_archive_directory,NULL,NULL);
7171

72-
EmitWarningsOnPlaceholders("basic_archive");
72+
MarkGUCPrefixReserved("basic_archive");
7373

7474
basic_archive_context=AllocSetContextCreate(TopMemoryContext,
7575
"basic_archive",

‎contrib/pg_prewarm/autoprewarm.c‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -137,7 +137,7 @@ _PG_init(void)
137137
NULL,
138138
NULL);
139139

140-
EmitWarningsOnPlaceholders("pg_prewarm");
140+
MarkGUCPrefixReserved("pg_prewarm");
141141

142142
RequestAddinShmemSpace(MAXALIGN(sizeof(AutoPrewarmSharedState)));
143143

‎contrib/pg_stat_statements/pg_stat_statements.c‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -437,7 +437,7 @@ _PG_init(void)
437437
NULL,
438438
NULL);
439439

440-
EmitWarningsOnPlaceholders("pg_stat_statements");
440+
MarkGUCPrefixReserved("pg_stat_statements");
441441

442442
/*
443443
* Request additional shared resources. (These are no-ops if we're not in

‎contrib/pg_trgm/trgm_op.c‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -101,7 +101,7 @@ _PG_init(void)
101101
NULL,
102102
NULL);
103103

104-
EmitWarningsOnPlaceholders("pg_trgm");
104+
MarkGUCPrefixReserved("pg_trgm");
105105
}
106106

107107
/*

‎contrib/postgres_fdw/option.c‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -538,5 +538,5 @@ _PG_init(void)
538538
NULL,
539539
NULL);
540540

541-
EmitWarningsOnPlaceholders("postgres_fdw");
541+
MarkGUCPrefixReserved("postgres_fdw");
542542
}

‎contrib/sepgsql/hooks.c‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -455,7 +455,7 @@ _PG_init(void)
455455
NULL,
456456
NULL);
457457

458-
EmitWarningsOnPlaceholders("sepgsql");
458+
MarkGUCPrefixReserved("sepgsql");
459459

460460
/* Initialize userspace access vector cache */
461461
sepgsql_avc_init();

‎src/backend/utils/misc/guc.c‎

Lines changed: 64 additions & 15 deletions
Original file line numberDiff line numberDiff line change
@@ -150,6 +150,8 @@ extern bool optimize_bounded_sort;
150150

151151
staticintGUC_check_errcode_value;
152152

153+
staticList*reserved_class_prefix=NIL;
154+
153155
/* global variables for check hook support */
154156
char*GUC_check_errmsg_string;
155157
char*GUC_check_errdetail_string;
@@ -5590,18 +5592,44 @@ find_option(const char *name, bool create_placeholders, bool skip_errors,
55905592
* doesn't contain a separator, don't assume that it was meant to be a
55915593
* placeholder.
55925594
*/
5593-
if (strchr(name,GUC_QUALIFIER_SEPARATOR)!=NULL)
5595+
constchar*sep=strchr(name,GUC_QUALIFIER_SEPARATOR);
5596+
5597+
if (sep!=NULL)
55945598
{
5595-
if (valid_custom_variable_name(name))
5596-
returnadd_placeholder_variable(name,elevel);
5597-
/* A special error message seems desirable here */
5598-
if (!skip_errors)
5599-
ereport(elevel,
5600-
(errcode(ERRCODE_INVALID_NAME),
5601-
errmsg("invalid configuration parameter name \"%s\"",
5602-
name),
5603-
errdetail("Custom parameter names must be two or more simple identifiers separated by dots.")));
5604-
returnNULL;
5599+
size_tclassLen=sep-name;
5600+
ListCell*lc;
5601+
5602+
/* The name must be syntactically acceptable ... */
5603+
if (!valid_custom_variable_name(name))
5604+
{
5605+
if (!skip_errors)
5606+
ereport(elevel,
5607+
(errcode(ERRCODE_INVALID_NAME),
5608+
errmsg("invalid configuration parameter name \"%s\"",
5609+
name),
5610+
errdetail("Custom parameter names must be two or more simple identifiers separated by dots.")));
5611+
returnNULL;
5612+
}
5613+
/* ... and it must not match any previously-reserved prefix */
5614+
foreach(lc,reserved_class_prefix)
5615+
{
5616+
constchar*rcprefix=lfirst(lc);
5617+
5618+
if (strlen(rcprefix)==classLen&&
5619+
strncmp(name,rcprefix,classLen)==0)
5620+
{
5621+
if (!skip_errors)
5622+
ereport(elevel,
5623+
(errcode(ERRCODE_INVALID_NAME),
5624+
errmsg("invalid configuration parameter name \"%s\"",
5625+
name),
5626+
errdetail("\"%s\" is a reserved prefix.",
5627+
rcprefix)));
5628+
returnNULL;
5629+
}
5630+
}
5631+
/* OK, create it */
5632+
returnadd_placeholder_variable(name,elevel);
56055633
}
56065634
}
56075635

@@ -9355,15 +9383,26 @@ DefineCustomEnumVariable(const char *name,
93559383
}
93569384

93579385
/*
9386+
* Mark the given GUC prefix as "reserved".
9387+
*
9388+
* This deletes any existing placeholders matching the prefix,
9389+
* and then prevents new ones from being created.
93589390
* Extensions should call this after they've defined all of their custom
93599391
* GUCs, to help catch misspelled config-file entries.
93609392
*/
93619393
void
9362-
EmitWarningsOnPlaceholders(constchar*className)
9394+
MarkGUCPrefixReserved(constchar*className)
93639395
{
93649396
intclassLen=strlen(className);
93659397
inti;
9398+
MemoryContextoldcontext;
93669399

9400+
/*
9401+
* Check for existing placeholders. We must actually remove invalid
9402+
* placeholders, else future parallel worker startups will fail. (We
9403+
* don't bother trying to free associated memory, since this shouldn't
9404+
* happen often.)
9405+
*/
93679406
for (i=0;i<num_guc_variables;i++)
93689407
{
93699408
structconfig_generic*var=guc_variables[i];
@@ -9373,11 +9412,21 @@ EmitWarningsOnPlaceholders(const char *className)
93739412
var->name[classLen]==GUC_QUALIFIER_SEPARATOR)
93749413
{
93759414
ereport(WARNING,
9376-
(errcode(ERRCODE_UNDEFINED_OBJECT),
9377-
errmsg("unrecognized configuration parameter \"%s\"",
9378-
var->name)));
9415+
(errcode(ERRCODE_INVALID_NAME),
9416+
errmsg("invalid configuration parameter name \"%s\", removing it",
9417+
var->name),
9418+
errdetail("\"%s\" is now a reserved prefix.",
9419+
className)));
9420+
num_guc_variables--;
9421+
memmove(&guc_variables[i],&guc_variables[i+1],
9422+
(num_guc_variables-i)*sizeof(structconfig_generic*));
93799423
}
93809424
}
9425+
9426+
/* And remember the name so we can prevent future mistakes. */
9427+
oldcontext=MemoryContextSwitchTo(TopMemoryContext);
9428+
reserved_class_prefix=lappend(reserved_class_prefix,pstrdup(className));
9429+
MemoryContextSwitchTo(oldcontext);
93819430
}
93829431

93839432

‎src/include/utils/guc.h‎

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -354,7 +354,10 @@ extern void DefineCustomEnumVariable(const char *name,
354354
GucEnumAssignHookassign_hook,
355355
GucShowHookshow_hook);
356356

357-
externvoidEmitWarningsOnPlaceholders(constchar*className);
357+
externvoidMarkGUCPrefixReserved(constchar*className);
358+
359+
/* old name for MarkGUCPrefixReserved, for backwards compatibility: */
360+
#defineEmitWarningsOnPlaceholders(className) MarkGUCPrefixReserved(className)
358361

359362
externconstchar*GetConfigOption(constchar*name,boolmissing_ok,
360363
boolrestrict_privileged);

0 commit comments

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