You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@trafficserver.apache.org by bc...@apache.org on 2015/04/17 21:34:13 UTC
[1/2] trafficserver git commit: TS-3530: HTTP/2 memory leak with
headers and more...
Repository: trafficserver
Updated Branches:
refs/heads/master 43a4086ca -> cd9c8a3b5
TS-3530: HTTP/2 memory leak with headers and more...
Project: http://git-wip-us.apache.org/repos/asf/trafficserver/repo
Commit: http://git-wip-us.apache.org/repos/asf/trafficserver/commit/bc2ae5a8
Tree: http://git-wip-us.apache.org/repos/asf/trafficserver/tree/bc2ae5a8
Diff: http://git-wip-us.apache.org/repos/asf/trafficserver/diff/bc2ae5a8
Branch: refs/heads/master
Commit: bc2ae5a8dbcfe75b60d419190d1ee7dadae26d68
Parents: 43a4086
Author: Bryan Call <bc...@apache.org>
Authored: Fri Apr 17 14:27:56 2015 -0500
Committer: Bryan Call <bc...@apache.org>
Committed: Fri Apr 17 14:27:56 2015 -0500
----------------------------------------------------------------------
proxy/http2/Http2ConnectionState.cc | 2 +-
proxy/http2/Http2ConnectionState.h | 2 +-
2 files changed, 2 insertions(+), 2 deletions(-)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/trafficserver/blob/bc2ae5a8/proxy/http2/Http2ConnectionState.cc
----------------------------------------------------------------------
diff --git a/proxy/http2/Http2ConnectionState.cc b/proxy/http2/Http2ConnectionState.cc
index 3322d6d..a63f4fd 100644
--- a/proxy/http2/Http2ConnectionState.cc
+++ b/proxy/http2/Http2ConnectionState.cc
@@ -653,7 +653,7 @@ Http2ConnectionState::main_event_handler(int event, void *edata)
if (error != HTTP2_ERROR_NO_ERROR) {
this->send_goaway_frame(last_streamid, error);
-
+ cleanup_streams();
// XXX We need to think a bit harder about how to coordinate the client session and the
// protocol connection. At this point, the protocol is shutting down, but there's no way
// to tell that to the client session. Perhaps this could be solved by implementing the
http://git-wip-us.apache.org/repos/asf/trafficserver/blob/bc2ae5a8/proxy/http2/Http2ConnectionState.h
----------------------------------------------------------------------
diff --git a/proxy/http2/Http2ConnectionState.h b/proxy/http2/Http2ConnectionState.h
index 8f33542..6a97690 100644
--- a/proxy/http2/Http2ConnectionState.h
+++ b/proxy/http2/Http2ConnectionState.h
@@ -105,7 +105,7 @@ public:
~Http2Stream()
{
- _req_header.clear();
+ _req_header.destroy();
if (_fetch_sm) {
_fetch_sm->ext_destroy();
[2/2] trafficserver git commit: TS-3530: Updated CHANGES
Posted by bc...@apache.org.
TS-3530: Updated CHANGES
Project: http://git-wip-us.apache.org/repos/asf/trafficserver/repo
Commit: http://git-wip-us.apache.org/repos/asf/trafficserver/commit/cd9c8a3b
Tree: http://git-wip-us.apache.org/repos/asf/trafficserver/tree/cd9c8a3b
Diff: http://git-wip-us.apache.org/repos/asf/trafficserver/diff/cd9c8a3b
Branch: refs/heads/master
Commit: cd9c8a3b517eb6c4039c19196f2ed48cca985f4f
Parents: bc2ae5a
Author: Bryan Call <bc...@apache.org>
Authored: Fri Apr 17 14:30:02 2015 -0500
Committer: Bryan Call <bc...@apache.org>
Committed: Fri Apr 17 14:31:15 2015 -0500
----------------------------------------------------------------------
CHANGES | 2 ++
1 file changed, 2 insertions(+)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/trafficserver/blob/cd9c8a3b/CHANGES
----------------------------------------------------------------------
diff --git a/CHANGES b/CHANGES
index 044cae6..3ec4999 100644
--- a/CHANGES
+++ b/CHANGES
@@ -1,6 +1,8 @@
-*- coding: utf-8 -*-
Changes with Apache Traffic Server 6.0.0
+ *) [TS-3530] HTTP/2 memory leak with headers and more...
+
*) [TS-3519] HTTP/2 seeing a lot of GOAWAY frames being sent
*) [TS-3520] Fix logging core dump due to invalid unmapped url length/buffer