[trafficserver] branch 7.1.x updated: coredump with server_session

[zw...@apache.org]

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

commit a40a89d3717a50fbe114d6ecf9605e1d43c9d355
Author: scw00 <61...@qq.com>
AuthorDate: Thu May 18 09:00:33 2017 +0800

    coredump with server_session
    
    (cherry picked from commit 1f8d00d35cd239ccc1544776e6bee4a7b85cfeb4)
---
 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 5a90847..2f3a8b5 100644
--- a/proxy/http/HttpSM.cc
+++ b/proxy/http/HttpSM.cc
@@ -3133,8 +3133,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
@@ -3163,6 +3162,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
['"commits@trafficserver.apache.org" <co...@trafficserver.apache.org>'].