This is an automated email from the ASF dual-hosted git repository.
shinrich pushed a commit to branch master
in repository https://git-dual.apache.org/repos/asf/trafficserver.git
The following commit(s) were added to refs/heads/master by this push:
new 1b09d40 TS-4899: Http2ClientSession object leaks.
1b09d40 is described below
commit 1b09d40b8f117923942ad951cd7c859811ec41a3
Author: Susan Hinrichs <[email protected]>
AuthorDate: Wed Sep 28 01:26:41 2016 +0000
TS-4899: Http2ClientSession object leaks.
---
proxy/http2/Http2ClientSession.cc | 3 ++-
proxy/http2/Http2ConnectionState.cc | 4 +---
2 files changed, 3 insertions(+), 4 deletions(-)
diff --git a/proxy/http2/Http2ClientSession.cc
b/proxy/http2/Http2ClientSession.cc
index fcb1179..eeac65e 100644
--- a/proxy/http2/Http2ClientSession.cc
+++ b/proxy/http2/Http2ClientSession.cc
@@ -313,7 +313,8 @@ Http2ClientSession::main_event_handler(int event, void
*edata)
case VC_EVENT_ERROR:
case VC_EVENT_EOS:
this->do_io_close();
- return 0;
+ retval = 0;
+ break;
case VC_EVENT_WRITE_READY:
retval = 0;
diff --git a/proxy/http2/Http2ConnectionState.cc
b/proxy/http2/Http2ConnectionState.cc
index 7fe5efe..212536e 100644
--- a/proxy/http2/Http2ConnectionState.cc
+++ b/proxy/http2/Http2ConnectionState.cc
@@ -807,9 +807,7 @@ Http2ConnectionState::main_event_handler(int event, void
*edata)
SCOPED_MUTEX_LOCK(lock, this->mutex, this_ethread());
send_data_frames_depends_on_priority();
_scheduled = false;
-
- return 0;
- }
+ } break;
// Parse received HTTP/2 frames
case HTTP2_SESSION_EVENT_RECV: {
--
To stop receiving notification emails like this one, please contact
['"[email protected]" <[email protected]>'].
