This is an automated email from the ASF dual-hosted git repository.

bcall pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/trafficserver.git


The following commit(s) were added to refs/heads/master by this push:
     new 60fa5c1  Fixes, I think, coverity issues 1379288
60fa5c1 is described below

commit 60fa5c10e2752b534f38356d5e9a885054f6ff4e
Author: Leif Hedstrom <zw...@apache.org>
AuthorDate: Thu Aug 10 11:00:12 2017 -0600

    Fixes, I think, coverity issues 1379288
---
 iocore/dns/DNS.cc | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/iocore/dns/DNS.cc b/iocore/dns/DNS.cc
index 4afb522..7bf7850 100644
--- a/iocore/dns/DNS.cc
+++ b/iocore/dns/DNS.cc
@@ -1238,7 +1238,7 @@ dns_result(DNSHandler *h, DNSEntry *e, HostEnt *ent, bool 
retry)
     }
   }
 
-  if (ent || !ent->good) {
+  if (ent && ent->good) {
     DNS_INCREMENT_DYN_STAT(dns_lookup_success_stat);
   } else {
     DNS_INCREMENT_DYN_STAT(dns_lookup_fail_stat);

-- 
To stop receiving notification emails like this one, please contact
['"commits@trafficserver.apache.org" <commits@trafficserver.apache.org>'].

Reply via email to