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  1f8d00d   coredump with server_session
1f8d00d is described below

commit 1f8d00d35cd239ccc1544776e6bee4a7b85cfeb4
Author: scw00 <[email protected]>
AuthorDate: Thu May 18 09:00:33 2017 +0800

    coredump with server_session
---
 proxy/http/HttpSM.cc | 9 +++++++--
 1 file changed, 7 insertions(+), 2 deletions(-)

diff --git a/proxy/http/HttpSM.cc b/proxy/http/HttpSM.cc
index 8e2b828..70b885f 100644
--- a/proxy/http/HttpSM.cc
+++ b/proxy/http/HttpSM.cc
@@ -3026,8 +3026,7 @@ HttpSM::tunnel_handler_server(int event, 
HttpTunnelProducer *p)
 
   if (close_connection) {
     p->vc->do_io_close();
-    server_session = nullptr; // Because p->vc == server_session
-    p->read_vio    = nullptr;
+    p->read_vio = nullptr;
     /* TS-1424: if we're outbound transparent and using the client
        source port for the outbound connection we must effectively
        propagate server closes back to the client. Part of that is
@@ -3056,6 +3055,12 @@ HttpSM::tunnel_handler_server(int event, 
HttpTunnelProducer *p)
     }
   }
 
+  // The server session has been released. Clean all pointer
+  server_entry->in_tunnel = true; // to avid cleaning in clenup_entry
+  vc_table.cleanup_entry(server_entry);
+  server_session = nullptr; // Because p->vc == server_session
+  server_entry   = nullptr;
+
   return 0;
 }
 

-- 
To stop receiving notification emails like this one, please contact
['"[email protected]" <[email protected]>'].

Reply via email to