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

Commitcb88f44

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 parent48e7971 commitcb88f44

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
@@ -6980,9 +6980,12 @@ set_config_option(const char *name, const char *value,
69806980
* Other changes might need to affect other workers, so forbid them.
69816981
*/
69826982
if (IsInParallelMode()&&changeVal&&action!=GUC_ACTION_SAVE)
6983+
{
69836984
ereport(elevel,
69846985
(errcode(ERRCODE_INVALID_TRANSACTION_STATE),
69856986
errmsg("cannot set parameters during a parallel operation")));
6987+
return-1;
6988+
}
69866989

69876990
record=find_option(name, true,elevel);
69886991
if (record==NULL)
@@ -7072,6 +7075,10 @@ set_config_option(const char *name, const char *value,
70727075
* backends. This is a tad klugy, but necessary because we
70737076
* don't re-read the config file during backend start.
70747077
*
7078+
* However, if changeVal is false then plow ahead anyway since
7079+
* we are trying to find out if the value is potentially good,
7080+
* not actually use it.
7081+
*
70757082
* In EXEC_BACKEND builds, this works differently: we load all
70767083
* non-default settings from the CONFIG_EXEC_PARAMS file
70777084
* during backend start. In that case we must accept
@@ -7082,7 +7089,7 @@ set_config_option(const char *name, const char *value,
70827089
* started it. is_reload will be true when either situation
70837090
* applies.
70847091
*/
7085-
if (IsUnderPostmaster&& !is_reload)
7092+
if (IsUnderPostmaster&&changeVal&&!is_reload)
70867093
return-1;
70877094
}
70887095
elseif (context!=PGC_POSTMASTER&&

0 commit comments

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