Commit 033b2343 authored by Robert Haas's avatar Robert Haas

Fix inadvertent semantics change in last patch to plug memory leaks.

Commit a5bca4ef accidentally changed
the semantics when the "skipping missing configuration file" is
emitted, because it forced OK to true instead of leaving the value
untouched.

Spotted by Tom Lane.
parent 5709b8ac
...@@ -476,7 +476,6 @@ ParseConfigFile(const char *config_file, const char *calling_file, bool strict, ...@@ -476,7 +476,6 @@ ParseConfigFile(const char *config_file, const char *calling_file, bool strict,
ereport(LOG, ereport(LOG,
(errmsg("skipping missing configuration file \"%s\"", (errmsg("skipping missing configuration file \"%s\"",
abs_path))); abs_path)));
OK = true;
goto cleanup; goto cleanup;
} }
......
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