Fix assertion failure when using -X and lock-file in configuration

When 'lock-file <lockfile1>' was used in configuration at the same time
as using `-X <lockfile2>` in `named` invocation, there was an invalid
logic that would lead to a double isc_mem_strdup() call on the
<lockfile2> value.

Skip the second allocation if `lock-file` is being used in
configuration, so the <lockfile2> is used only single time.

diff --git a/bin/named/server.c b/bin/named/server.c
index 9333d206c85467a8eff53ea5709fd2bd8fb787d0..e2d49b7341dd5c0cf9e823d0cad245363fed1b8f 100644 (file)
--- a/bin/named/server.c
+++ b/bin/named/server.c
@@ -8187,10 +8187,7 @@ check_lockfile(named_server_t *server, const cfg_obj_t *config,
                if (server->lockfile == NULL) {
                        return (ISC_R_NOMEMORY);
                }
-       }
-
-       if (named_g_forcelock && named_g_defaultlockfile != NULL) {
-               INSIST(server->lockfile == NULL);
+       } else if (named_g_forcelock && named_g_defaultlockfile != NULL) {
                server->lockfile = isc_mem_strdup(server->mctx,
                                                  named_g_defaultlockfile);
        }