[trafficserver] branch 7.1.x updated: Acquire mutex lock of Http2Stream before starting new transaction

[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 b696e08  Acquire mutex lock of Http2Stream before starting new transaction
b696e08 is described below

commit b696e08ea0284897618ef87b5a33b14899272109
Author: Masaori Koshiba <ma...@apache.org>
AuthorDate: Fri Apr 20 11:35:40 2018 +0900

    Acquire mutex lock of Http2Stream before starting new transaction
    
    (cherry picked from commit 8a0129b6668ca676c8f6d8f35484991df56afda1)
---
 proxy/http2/Http2ConnectionState.cc | 4 ++++
 1 file changed, 4 insertions(+)

diff --git a/proxy/http2/Http2ConnectionState.cc b/proxy/http2/Http2ConnectionState.cc
index a83914d..e45e624 100644
--- a/proxy/http2/Http2ConnectionState.cc
+++ b/proxy/http2/Http2ConnectionState.cc
@@ -336,6 +336,7 @@ rcv_headers_frame(Http2ConnectionState &cstate, const Http2Frame &frame)
 
     // Set up the State Machine
     if (!empty_request) {
+      SCOPED_MUTEX_LOCK(stream_lock, stream->mutex, this_ethread());
       stream->new_transaction();
       // Send request header to SM
       stream->send_request(cstate);
@@ -810,6 +811,7 @@ rcv_continuation_frame(Http2ConnectionState &cstate, const Http2Frame &frame)
     }
 
     // Set up the State Machine
+    SCOPED_MUTEX_LOCK(stream_lock, stream->mutex, this_ethread());
     stream->new_transaction();
     // Send request header to SM
     stream->send_request(cstate);
@@ -1562,6 +1564,8 @@ Http2ConnectionState::send_push_promise_frame(Http2Stream *stream, URL &url, con
     h2_hdr.destroy();
     return;
   }
+
+  SCOPED_MUTEX_LOCK(stream_lock, stream->mutex, this_ethread());
   if (Http2::stream_priority_enabled) {
     Http2DependencyTree::Node *node = this->dependency_tree->find(id);
     if (node != nullptr) {

-- 
To stop receiving notification emails like this one, please contact
zwoop@apache.org.