]> git.ipfire.org Git - thirdparty/bind9.git/commitdiff
'dispatch' must be non NULL, remove test.
authorMark Andrews <marka@isc.org>
Wed, 5 Feb 2020 05:53:43 +0000 (16:53 +1100)
committerMark Andrews <marka@isc.org>
Wed, 5 Feb 2020 07:37:17 +0000 (18:37 +1100)
10067 cleanup:

CID 1452683 (#1 of 1): Dereference before null check (REVERSE_INULL)
check_after_deref: Null-checking dispatch suggests that it
may be null, but it has already been dereferenced on all
paths leading to the check.

10068        if (dispatch != NULL)
10069                isc_mem_put(server->mctx, dispatch, sizeof(*dispatch));

bin/named/server.c

index 3acd9ad0e64f153b037e354b80433c442bf9adb5..6a818a31493d731fd1f89ec9f737fd0af745d541 100644 (file)
@@ -10065,8 +10065,7 @@ named_add_reserved_dispatch(named_server_t *server,
        return;
 
  cleanup:
-       if (dispatch != NULL)
-               isc_mem_put(server->mctx, dispatch, sizeof(*dispatch));
+       isc_mem_put(server->mctx, dispatch, sizeof(*dispatch));
        isc_sockaddr_format(addr, addrbuf, sizeof(addrbuf));
        isc_log_write(named_g_lctx, NAMED_LOGCATEGORY_GENERAL,
                      NAMED_LOGMODULE_SERVER, ISC_LOG_WARNING,