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 2017/10/31 15:03:58 UTC

[trafficserver] branch 7.1.x updated: fix 2707, assign server_session to nullptr after do `chain_abort_all`

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

zwoop pushed a commit to branch 7.1.x
in repository https://gitbox.apache.org/repos/asf/trafficserver.git


The following commit(s) were added to refs/heads/7.1.x by this push:
     new bb218fb  fix 2707, assign server_session to nullptr after do `chain_abort_all`
bb218fb is described below

commit bb218fbe1e318ccee2ba9963e3fafd0e675fdadd
Author: scw00 <sc...@apache.org>
AuthorDate: Wed Oct 25 08:32:24 2017 +0800

    fix 2707, assign server_session to nullptr after do `chain_abort_all`
    
    (cherry picked from commit 0be3931ee4ac9592c2983621634aac9f4d58a3b1)
---
 proxy/http/HttpSM.cc | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

diff --git a/proxy/http/HttpSM.cc b/proxy/http/HttpSM.cc
index 1f2f986..80fab37 100644
--- a/proxy/http/HttpSM.cc
+++ b/proxy/http/HttpSM.cc
@@ -3524,6 +3524,7 @@ HttpSM::tunnel_handler_post_ua(int event, HttpTunnelProducer *p)
       DebugSM("http_tunnel", "send 408 response to client to vc %p, tunnel vc %p", ua_session->get_netvc(), p->vc);
 
       tunnel.chain_abort_all(p);
+      server_session = nullptr;
       // Reset the inactivity timeout, otherwise the InactivityCop will callback again in the next second.
       ua_session->set_inactivity_timeout(HRTIME_SECONDS(t_state.txn_conf->transaction_no_activity_timeout_in));
       // if it is active timeout case, we need to give another chance to send 408 response;
@@ -3545,7 +3546,8 @@ HttpSM::tunnel_handler_post_ua(int event, HttpTunnelProducer *p)
     set_ua_abort(HttpTransact::ABORTED, event);
 
     tunnel.chain_abort_all(p);
-    p->read_vio = nullptr;
+    server_session = nullptr;
+    p->read_vio    = nullptr;
     p->vc->do_io_close(EHTTP_ERROR);
 
     // the in_tunnel status on both the ua & and

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