Commit b8646012 authored by Tom Lane's avatar Tom Lane

Tweak newly added set_config_sourcefile() so that the target record

isn't left corrupt if guc_strdup should fail.
parent f867339c
...@@ -10,7 +10,7 @@ ...@@ -10,7 +10,7 @@
* Written by Peter Eisentraut <peter_e@gmx.net>. * Written by Peter Eisentraut <peter_e@gmx.net>.
* *
* IDENTIFICATION * IDENTIFICATION
* $PostgreSQL: pgsql/src/backend/utils/misc/guc.c,v 1.471 2008/09/10 18:09:19 alvherre Exp $ * $PostgreSQL: pgsql/src/backend/utils/misc/guc.c,v 1.472 2008/09/10 19:16:22 tgl Exp $
* *
*-------------------------------------------------------------------- *--------------------------------------------------------------------
*/ */
...@@ -5128,9 +5128,10 @@ set_config_sourcefile(const char *name, char *sourcefile, int sourceline) ...@@ -5128,9 +5128,10 @@ set_config_sourcefile(const char *name, char *sourcefile, int sourceline)
if (record == NULL) if (record == NULL)
elog(ERROR, "unrecognized configuration parameter \"%s\"", name); elog(ERROR, "unrecognized configuration parameter \"%s\"", name);
sourcefile = guc_strdup(elevel, sourcefile);
if (record->sourcefile) if (record->sourcefile)
free(record->sourcefile); free(record->sourcefile);
record->sourcefile = guc_strdup(elevel, sourcefile); record->sourcefile = sourcefile;
record->sourceline = sourceline; record->sourceline = sourceline;
} }
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment