diff --git a/CHANGES b/CHANGES index 0c77506969..6377ddedb2 100644 --- a/CHANGES +++ b/CHANGES @@ -1,3 +1,6 @@ +6275. [bug] Fix assertion failure when using lock-file configuration + option together -X argument to named. [GL #4386] + 6274. [bug] The 'lock-file' file was being removed when it shouldn't have been making it ineffective if named was started 3 or more times. [GL #4387] diff --git a/bin/named/server.c b/bin/named/server.c index 2f21fc5213..3295082f9d 100644 --- a/bin/named/server.c +++ b/bin/named/server.c @@ -8443,10 +8443,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); } diff --git a/doc/notes/notes-current.rst b/doc/notes/notes-current.rst index b857f117a3..4c1c950fc1 100644 --- a/doc/notes/notes-current.rst +++ b/doc/notes/notes-current.rst @@ -43,6 +43,9 @@ Bug Fixes have been making it ineffective if named was started 3 or more times. :gl:`#4387` +- When :any:`lock-file` was used at the same time as :option:`named -X`, the + assertion failure would be triggered. This has been fixed. :gl:`#4386` + Known Issues ~~~~~~~~~~~~