You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@trafficserver.apache.org by zw...@apache.org on 2017/12/22 20:42:57 UTC
[trafficserver] branch master updated: Fix race condition during
testing rc2.
This is an automated email from the ASF dual-hosted git repository.
zwoop 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 a3e7eb6 Fix race condition during testing rc2.
a3e7eb6 is described below
commit a3e7eb6f133c78479bc7c19661112a963369471c
Author: Zeyuan Yu <ze...@gmail.com>
AuthorDate: Thu Dec 21 22:08:53 2017 +0000
Fix race condition during testing rc2.
---
proxy/http/HttpSM.cc | 8 +++++++-
1 file changed, 7 insertions(+), 1 deletion(-)
diff --git a/proxy/http/HttpSM.cc b/proxy/http/HttpSM.cc
index 292b6fd..f84f023 100644
--- a/proxy/http/HttpSM.cc
+++ b/proxy/http/HttpSM.cc
@@ -5480,6 +5480,12 @@ HttpSM::handle_server_setup_error(int event, void *data)
// if (vio->op == VIO::WRITE && vio->ndone == 0) {
if (server_entry->write_vio && server_entry->write_vio->nbytes > 0 && server_entry->write_vio->ndone == 0) {
t_state.current.state = HttpTransact::CONNECTION_ERROR;
+ if (server_entry) {
+ ink_assert(server_entry->vc_type == HTTP_SERVER_VC);
+ vc_table.cleanup_entry(server_entry);
+ server_entry = nullptr;
+ server_session = nullptr;
+ }
} else {
t_state.current.state = HttpTransact::INACTIVE_TIMEOUT;
}
@@ -5489,7 +5495,7 @@ HttpSM::handle_server_setup_error(int event, void *data)
}
// Closedown server connection and deallocate buffers
- ink_assert(server_entry->in_tunnel == false);
+ ink_assert(!server_entry || server_entry->in_tunnel == false);
// if we are waiting on a plugin callout for
// HTTP_API_SEND_REQUEST_HDR defer calling transact until
--
To stop receiving notification emails like this one, please contact
['"commits@trafficserver.apache.org" <co...@trafficserver.apache.org>'].