[trafficserver] branch master updated: TS-5006: Fix CID 1356975

[zw...@apache.org]

This is an automated email from the ASF dual-hosted git repository.

zwoop pushed a commit to branch master
in repository https://git-dual.apache.org/repos/asf/trafficserver.git

The following commit(s) were added to refs/heads/master by this push:
       new  48eda68   TS-5006: Fix CID 1356975
48eda68 is described below

commit 48eda68822da82ebf687d96448438941a15bf62a
Author: Phil Sorber <so...@apache.org>
AuthorDate: Wed Oct 26 11:56:09 2016 -0600

    TS-5006: Fix CID 1356975
    
    ua_session can not be NULL so checking for it here only confuses coverity.
---
 proxy/http/HttpSM.cc | 8 +++-----
 1 file changed, 3 insertions(+), 5 deletions(-)

diff --git a/proxy/http/HttpSM.cc b/proxy/http/HttpSM.cc
index 3841acf..b7c3d8a 100644
--- a/proxy/http/HttpSM.cc
+++ b/proxy/http/HttpSM.cc
@@ -3218,11 +3218,9 @@ HttpSM::tunnel_handler_ua(int event, HttpTunnelConsumer *c)
   case VC_EVENT_ACTIVE_TIMEOUT:
   case VC_EVENT_ERROR:
 
-    if (ua_session != NULL) {
-      // The user agent died or aborted.  Check to
-      //  see if we should setup a background fill
-      set_ua_abort(HttpTransact::ABORTED, event);
-    }
+    // The user agent died or aborted.  Check to
+    //  see if we should setup a background fill
+    set_ua_abort(HttpTransact::ABORTED, event);
 
     if (is_bg_fill_necessary(c)) {
       DebugSM("http", "[%" PRId64 "] Initiating background fill", sm_id);

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