[trafficserver] branch master updated: coverity 1022121 : remove check

[bc...@apache.org]

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  92a39c7   coverity 1022121 : remove check
92a39c7 is described below

commit 92a39c7a61b0eba227a463b1a0b25611870a2f66
Author: Jason Kenny <dr...@live.com>
AuthorDate: Thu May 11 10:40:33 2017 -0400

    coverity 1022121 : remove check
    
    coverity 1196433 : remove dead code
---
 iocore/cache/CacheHosting.cc | 9 ++-------
 1 file changed, 2 insertions(+), 7 deletions(-)

diff --git a/iocore/cache/CacheHosting.cc b/iocore/cache/CacheHosting.cc
index ac67e0a..24f102e 100644
--- a/iocore/cache/CacheHosting.cc
+++ b/iocore/cache/CacheHosting.cc
@@ -320,10 +320,6 @@ CacheHostTable::BuildTableFromString(const char *config_file_path, char *file_bu
     if (gen_host_rec.Init(type)) {
       Warning("Problems encountered while initializing the Generic Volume");
     }
-
-    if (first != nullptr) {
-      ats_free(first);
-    }
     return 0;
   }
 
@@ -532,9 +528,8 @@ CacheHostRecord::Init(matcher_line *line_info, CacheType typ)
         }
         s++;
       }
-      if (val != nullptr) {
-        ats_free(val);
-      }
+      ats_free(val);
+
       break;
     }
 

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