[trafficserver] 01/02: TS-3123: Missing NULL check and failing clang-analyzer

[so...@apache.org]

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

sorber pushed a commit to branch 6.2.x
in repository https://git-dual.apache.org/repos/asf/trafficserver.git

commit bf91c3c7eec07eb651458f35a5e326a667ebe8ee
Author: Bryan Call <bc...@apache.org>
AuthorDate: Mon May 23 17:11:04 2016 -0700

    TS-3123: Missing NULL check and failing clang-analyzer
    
    (cherry picked from commit 006d63653f9bfd65a041adee7d4c7b2d43737428)
---
 proxy/http/HttpTransact.cc | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

diff --git a/proxy/http/HttpTransact.cc b/proxy/http/HttpTransact.cc
index 87e1771..aa4812e 100644
--- a/proxy/http/HttpTransact.cc
+++ b/proxy/http/HttpTransact.cc
@@ -917,7 +917,9 @@ HttpTransact::EndRemapRequest(State *s)
 
 done:
   // We now set the active-timeout again, since it might have been changed as part of the remap rules.
-  s->state_machine->ua_session->get_netvc()->set_active_timeout(HRTIME_SECONDS(s->txn_conf->transaction_active_timeout_in));
+  if (s->state_machine->ua_session) {
+    s->state_machine->ua_session->get_netvc()->set_active_timeout(HRTIME_SECONDS(s->txn_conf->transaction_active_timeout_in));
+  }
 
   if (is_debug_tag_set("http_chdr_describe") || is_debug_tag_set("http_trans") || is_debug_tag_set("url_rewrite")) {
     DebugTxn("http_trans", "After Remapping:");

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