You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@trafficserver.apache.org by zw...@apache.org on 2015/01/03 21:30:28 UTC

[2/2] trafficserver git commit: Remove a test that's always true.

Remove a test that's always true.

Coverity CID #1261584


Project: http://git-wip-us.apache.org/repos/asf/trafficserver/repo
Commit: http://git-wip-us.apache.org/repos/asf/trafficserver/commit/4a310165
Tree: http://git-wip-us.apache.org/repos/asf/trafficserver/tree/4a310165
Diff: http://git-wip-us.apache.org/repos/asf/trafficserver/diff/4a310165

Branch: refs/heads/master
Commit: 4a310165820dc59d838ce89cbc7dd16f3169d634
Parents: 1c42c55
Author: Leif Hedstrom <zw...@apache.org>
Authored: Sat Jan 3 13:24:02 2015 -0700
Committer: Leif Hedstrom <zw...@apache.org>
Committed: Sat Jan 3 13:24:02 2015 -0700

----------------------------------------------------------------------
 iocore/cache/CacheHosting.cc | 8 +++-----
 1 file changed, 3 insertions(+), 5 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/trafficserver/blob/4a310165/iocore/cache/CacheHosting.cc
----------------------------------------------------------------------
diff --git a/iocore/cache/CacheHosting.cc b/iocore/cache/CacheHosting.cc
index 3c7395b..ad65061 100644
--- a/iocore/cache/CacheHosting.cc
+++ b/iocore/cache/CacheHosting.cc
@@ -797,11 +797,9 @@ ConfigVolumes::BuildListFromString(char *config_file_path, char *file_buf)
           total += size;
           if (size > 100 || total > 100) {
             state = INK_ERROR_VOLUME;
-            if (state == INK_ERROR_VOLUME || *tmp) {
-              RecSignalWarning(REC_SIGNAL_CONFIG_ERROR,
-                       "Total volume size added upto more than 100 percent,No volumes created");
-              break;
-            }
+            RecSignalWarning(REC_SIGNAL_CONFIG_ERROR,
+                             "Total volume size added upto more than 100 percent, No volumes created");
+            break;
           }
           //ends here
           in_percent = 1;