From 44b08521ef11e550ec5edea7378458aad8f2a89f Mon Sep 17 00:00:00 2001 From: Mark Andrews Date: Wed, 5 Feb 2020 16:41:03 +1100 Subject: [PATCH] 'tql' must be non NULL, remove test. 255 flag_fail: 256 /* get rid of what was build of the query list */ CID 1452697 (#1 of 1): Dereference before null check (REVERSE_INULL) check_after_deref: Null-checking tql suggests that it may be null, but it has already been dereferenced on all paths leading to the check. 257 if (tql != NULL) 258 destroy_querylist(mctx, &tql); --- contrib/dlz/drivers/sdlz_helper.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/contrib/dlz/drivers/sdlz_helper.c b/contrib/dlz/drivers/sdlz_helper.c index 889a8a6ee8..6eb6629245 100644 --- a/contrib/dlz/drivers/sdlz_helper.c +++ b/contrib/dlz/drivers/sdlz_helper.c @@ -254,8 +254,7 @@ build_querylist(isc_mem_t *mctx, const char *query_str, char **zone, flag_fail: /* get rid of what was build of the query list */ - if (tql != NULL) - destroy_querylist(mctx, &tql); + destroy_querylist(mctx, &tql); return result; }