[trafficserver] branch 7.1.x updated: Release the mutex after the events are cancelled

[zwoop]

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 a1f86f6  Release the mutex after the events are cancelled
a1f86f6 is described below

commit a1f86f6ee3ff23755518b2745663d903277dd77f
Author: Vijay Mamidi <vijayabhaskar_mam...@yahoo.com>
AuthorDate: Sat Nov 17 08:53:26 2018 +0900

    Release the mutex after the events are cancelled
---
 proxy/http2/Http2ConnectionState.h | 5 ++++-
 1 file changed, 4 insertions(+), 1 deletion(-)

diff --git a/proxy/http2/Http2ConnectionState.h 
b/proxy/http2/Http2ConnectionState.h
index 2410c61..f3b4cc3 100644
--- a/proxy/http2/Http2ConnectionState.h
+++ b/proxy/http2/Http2ConnectionState.h
@@ -154,15 +154,18 @@ public:
   {
     cleanup_streams();
 
-    mutex = NULL; // magic happens - assigning to NULL 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) {
       fini_event->cancel();
     }
+    mutex = NULL; // magic happens - assigning to NULL frees the ProxyMutex
   }
 
   // Event handlers