This is an automated email from the ASF dual-hosted git repository. zwoop pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/trafficserver.git
The following commit(s) were added to refs/heads/master by this push: new 0be3931 fix 2707, assign server_session to nullptr after do `chain_abort_all` 0be3931 is described below commit 0be3931ee4ac9592c2983621634aac9f4d58a3b1 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` --- 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 195d794..faaf19a 100644 --- a/proxy/http/HttpSM.cc +++ b/proxy/http/HttpSM.cc @@ -3387,6 +3387,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; @@ -3408,7 +3409,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" <commits@trafficserver.apache.org>'].