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

Commitea61b1c

Browse files
committed
Fix failure to verify PGC_[SU_]BACKEND GUCs in pg_file_settings view.
set_config_option() bails out early if it detects that the option tobe set is PGC_BACKEND or PGC_SU_BACKEND class and we're reading theconfig file in a postmaster child; we don't want to apply any newvalue in such a case. That's fine as far as it goes, but it failsto consider the requirements of the pg_file_settings view: for that,we need to check validity of the value even though we have nointention to apply it. Because we didn't, even very silly valuesfor affected GUCs would be reported as valid by the view. Thereare only half a dozen such GUCs, which perhaps explains why thisgot overlooked for so long.Fix by continuing when changeVal is false; this parallels the logicin some other early-exit paths.Also, the check added by commit924bcf4 to prevent GUC changes inparallel workers seems a few bricks shy of a load: it's evidentlyassuming that ereport(elevel, ...) won't return. Make sure webail out if it does. The lack of trouble reports suggests thatthis is only a latent bug, i.e. parallel workers don't actuallyreach here with elevel < ERROR. (Per the code coverage report,we never reach here at all in the regression suite.) But we clearlydon't want to risk proceeding if that does happen.Per report from Rıdvan Korkmaz. These are ancient bugs, so back-patchto all supported branches.Discussion:https://postgr.es/m/2089235.1703617353@sss.pgh.pa.us
1 parent1613e2f commitea61b1c

File tree

1 file changed

+8
-1
lines changed
  • src/backend/utils/misc

1 file changed

+8
-1
lines changed

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

Lines changed: 8 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -6749,9 +6749,12 @@ set_config_option(const char *name, const char *value,
67496749
* Other changes might need to affect other workers, so forbid them.
67506750
*/
67516751
if (IsInParallelMode()&&changeVal&&action!=GUC_ACTION_SAVE)
6752+
{
67526753
ereport(elevel,
67536754
(errcode(ERRCODE_INVALID_TRANSACTION_STATE),
67546755
errmsg("cannot set parameters during a parallel operation")));
6756+
return-1;
6757+
}
67556758

67566759
record=find_option(name, true,elevel);
67576760
if (record==NULL)
@@ -6841,6 +6844,10 @@ set_config_option(const char *name, const char *value,
68416844
* backends. This is a tad klugy, but necessary because we
68426845
* don't re-read the config file during backend start.
68436846
*
6847+
* However, if changeVal is false then plow ahead anyway since
6848+
* we are trying to find out if the value is potentially good,
6849+
* not actually use it.
6850+
*
68446851
* In EXEC_BACKEND builds, this works differently: we load all
68456852
* non-default settings from the CONFIG_EXEC_PARAMS file
68466853
* during backend start. In that case we must accept
@@ -6851,7 +6858,7 @@ set_config_option(const char *name, const char *value,
68516858
* started it. is_reload will be true when either situation
68526859
* applies.
68536860
*/
6854-
if (IsUnderPostmaster&& !is_reload)
6861+
if (IsUnderPostmaster&&changeVal&&!is_reload)
68556862
return-1;
68566863
}
68576864
elseif (context!=PGC_POSTMASTER&&

0 commit comments

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