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

Commit a5bca4ef034f71175d46462963af2329d22068c2 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.

diff --git a/src/backend/utils/misc/guc-file.l b/src/backend/utils/misc/guc-file.l
index a5b9d6ac77cc2c947e64f9c826400d70bb32989d..e815a9bef9af1fe902f450a874a2d65633e16876 100644 (file)
--- a/src/backend/utils/misc/guc-file.l
+++ b/src/backend/utils/misc/guc-file.l
@@ -476,7 +476,6 @@ ParseConfigFile(const char *config_file, const char *calling_file, bool strict,
                ereport(LOG,
                                (errmsg("skipping missing configuration file \"%s\"",
                                                abs_path)));
-               OK = true;
                goto cleanup;
        }