From dfbfb50f6409a4c380f233706402ded243155392 Mon Sep 17 00:00:00 2001 From: Jason Ish Date: Mon, 16 Jan 2017 10:31:01 -0600 Subject: [PATCH] dns (tcp) - fix coverity cid 1374307 CID 1374307 (#1 of 1): Dereference before null check (REVERSE_INULL) check_after_deref: Null-checking dns_state suggests that it may be null, but it has already been dereferenced on all paths leading to the check. 317 if (dns_state != NULL && f != NULL) { 318 dns_state->last_resp = f->lastts; 319 } --- src/app-layer-dns-udp.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/app-layer-dns-udp.c b/src/app-layer-dns-udp.c index 6a2523454d..d8ee37d30f 100644 --- a/src/app-layer-dns-udp.c +++ b/src/app-layer-dns-udp.c @@ -314,7 +314,7 @@ static int DNSUDPResponseParse(Flow *f, void *dstate, tx->replied = 1; } - if (dns_state != NULL && f != NULL) { + if (f != NULL) { dns_state->last_resp = f->lastts; } SCReturnInt(1);