This is an automated email from the ASF dual-hosted git repository.
masaori 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 8a0129b Acquire mutex lock of Http2Stream before starting new
transaction
8a0129b is described below
commit 8a0129b6668ca676c8f6d8f35484991df56afda1
Author: Masaori Koshiba <[email protected]>
AuthorDate: Fri Apr 20 11:35:40 2018 +0900
Acquire mutex lock of Http2Stream before starting new transaction
---
proxy/http2/Http2ConnectionState.cc | 4 ++++
1 file changed, 4 insertions(+)
diff --git a/proxy/http2/Http2ConnectionState.cc
b/proxy/http2/Http2ConnectionState.cc
index 7c16939..a83d397 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);
@@ -1590,6 +1592,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
[email protected].
