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