From: Aram Sargsyan Date: Mon, 6 Jun 2022 15:12:46 +0000 (+0000) Subject: Remove unneded NULL-checking X-Git-Tag: v9.19.3~62^2 X-Git-Url: http://git.ipfire.org/gitweb/?a=commitdiff_plain;h=b7cb73160bfb635f90f4d720bdd139ededfcc296;p=thirdparty%2Fbind9.git Remove unneded NULL-checking Fix an issue reported by Coverity by removing the unneded check. *** CID 352554: Null pointer dereferences (REVERSE_INULL) /bin/dig/dighost.c: 3056 in start_tcp() 3050 3051 if (ISC_LINK_LINKED(query, link)) { 3052 next = ISC_LIST_NEXT(query, link); 3053 } else { 3054 next = NULL; 3055 } >>> CID 352554: Null pointer dereferences (REVERSE_INULL) >>> Null-checking "connectquery" suggests that it may be null, but it has already been dereferenced on all paths leading to the check. 3056 if (connectquery != NULL) { 3057 query_detach(&connectquery); 3058 } 3059 query_detach(&query); 3060 if (next == NULL) { 3061 clear_current_lookup(); --- diff --git a/bin/dig/dighost.c b/bin/dig/dighost.c index 4a2e88fecf4..40d004c1ef5 100644 --- a/bin/dig/dighost.c +++ b/bin/dig/dighost.c @@ -3053,9 +3053,7 @@ failure_tls: } else { next = NULL; } - if (connectquery != NULL) { - query_detach(&connectquery); - } + query_detach(&connectquery); query_detach(&query); if (next == NULL) { clear_current_lookup();