[trafficserver] branch master updated: Clang Analyzer: Fix IpMap.cc false positives. #2

[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 b55a9fd  Clang Analyzer: Fix IpMap.cc false positives. #2
b55a9fd is described below

commit b55a9fda49d29003d1c81ccced54f55841a03a91
Author: Oknet Xu <xu...@skyguard.com.cn>
AuthorDate: Thu May 9 19:31:25 2019 +0800

    Clang Analyzer: Fix IpMap.cc false positives. #2
---
 src/tscore/IpMap.cc | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/src/tscore/IpMap.cc b/src/tscore/IpMap.cc
index 89f1143..87ab278 100644
--- a/src/tscore/IpMap.cc
+++ b/src/tscore/IpMap.cc
@@ -384,9 +384,9 @@ namespace detail
           if (n->_max <= max) {
 // next range is covered, so we can remove and continue.
 #if defined(__clang_analyzer__)
-            ink_assert(x != n)
+            ink_assert(x != n);
 #endif
-              this->remove(n);
+            this->remove(n);
             n = next(x);
           } else if (n->_min <= max_plus1) {
             // Overlap or adjacent with larger max - absorb and finish.