mirror of
https://github.com/nmap/nmap.git
synced 2026-01-01 20:39:02 +00:00
fixing a bug in setState() in traceroute.cc. A || was used instead of && which caused it to always evaluate true -- Coverity CID 23
This commit is contained in:
@@ -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);
|
||||
|
||||
Reference in New Issue
Block a user