[trafficserver] branch master updated: release the mutex after the events are cancelled and sessions are destroyed.

[vmamidi]

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

vmamidi 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 a639581  release the mutex after the events are cancelled and sessions 
are destroyed.
a639581 is described below

commit a639581c85d9b9c9dcb728b03e86efc280b303c3
Author: Vijay Mamidi <vijayabhaskar_mam...@yahoo.com>
AuthorDate: Mon Nov 12 15:08:52 2018 +0900

    release the mutex after the events are cancelled and sessions are destroyed.
---
 proxy/http2/Http2ConnectionState.h | 6 +++++-
 1 file changed, 5 insertions(+), 1 deletion(-)

diff --git a/proxy/http2/Http2ConnectionState.h 
b/proxy/http2/Http2ConnectionState.h
index bb2a2b2..ef94a93 100644
--- a/proxy/http2/Http2ConnectionState.h
+++ b/proxy/http2/Http2ConnectionState.h
@@ -141,10 +141,12 @@ public:
     }
     cleanup_streams();
 
-    mutex = nullptr; // magic happens - assigning to nullptr frees the 
ProxyMutex
     delete local_hpack_handle;
+    local_hpack_handle = nullptr;
     delete remote_hpack_handle;
+    remote_hpack_handle = nullptr;
     delete dependency_tree;
+    dependency_tree  = nullptr;
     this->ua_session = nullptr;
 
     if (fini_event) {
@@ -153,6 +155,8 @@ public:
     if (zombie_event) {
       zombie_event->cancel();
     }
+    // release the mutex after the events are cancelled and sessions are 
destroyed.
+    mutex = nullptr; // magic happens - assigning to nullptr frees the 
ProxyMutex
   }
 
   // Event handlers