From 01571cee0000ed7886dfa6ec33894da1da5ee4e2 Mon Sep 17 00:00:00 2001
From: kris <kris@e0a8ed71-7df4-0310-8962-fdc924857419>
Date: Thu, 15 Nov 2007 22:25:00 +0000
Subject: [PATCH] fixing a bug in setState() in traceroute.cc.  A || was used
 instead of && which caused it to always evaluate true -- Coverity CID 23

---
 traceroute.cc | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/traceroute.cc b/traceroute.cc
index b1abc8222..61fd3bf7e 100644
--- a/traceroute.cc
+++ b/traceroute.cc
@@ -1279,7 +1279,7 @@ void TraceGroup::consolidateHops () {
 
 u8
 TraceGroup::setState (u8 state) {
-    if (state <= G_FINISH || state >= G_OK)
+    if (state <= G_FINISH && state >= G_OK)
         this->state = state;
     else if (o.debugging)
         log_write (LOG_STDOUT, "%s: invalid tracegroup state %d\n", IPStr (), state);