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 2018/09/12 18:11:05 UTC

[trafficserver] branch 8.0.x updated (9305278 -> ad43863)

This is an automated email from the ASF dual-hosted git repository.

bcall pushed a change to branch 8.0.x
in repository https://gitbox.apache.org/repos/asf/trafficserver.git.


    from 9305278  Disable the HttpSM half open logic if the underlying transport is TLS
     new aaa9d0c  Fixed error getting h2 HEADERS frame after stream is closed
     new ad43863  Fix link error on macOS

The 2 revisions listed above as "new" are entirely new to this
repository and will be described in separate emails.  The revisions
listed as "add" were already present in the repository and have only
been added to this reference.


Summary of changes:
 proxy/http2/Http2ConnectionState.cc | 5 +++++
 src/tscore/Makefile.am              | 1 +
 2 files changed, 6 insertions(+)


[trafficserver] 02/02: Fix link error on macOS

Posted by bc...@apache.org.
This is an automated email from the ASF dual-hosted git repository.

bcall pushed a commit to branch 8.0.x
in repository https://gitbox.apache.org/repos/asf/trafficserver.git

commit ad43863e9a4c8c27b90c10d081578f91e0e29868
Author: Masaori Koshiba <ma...@apache.org>
AuthorDate: Wed Sep 12 14:54:05 2018 +0900

    Fix link error on macOS
    
    (cherry picked from commit 23b8d5b56a6edb1f31a1d2d3dfecd7e33ac7149c)
---
 src/tscore/Makefile.am | 1 +
 1 file changed, 1 insertion(+)

diff --git a/src/tscore/Makefile.am b/src/tscore/Makefile.am
index 65d6dbe..25b7ecd 100644
--- a/src/tscore/Makefile.am
+++ b/src/tscore/Makefile.am
@@ -38,6 +38,7 @@ AM_CPPFLAGS += \
 
 libtscore_la_LDFLAGS = -no-undefined -version-info @TS_LIBTOOL_VERSION@ @YAMLCPP_LDFLAGS@
 libtscore_la_LIBADD = \
+	$(top_builddir)/src/tscpp/util/libtscpputil.la \
 	@HWLOC_LIBS@ \
 	@LIBOBJS@ \
 	@LIBPCRE@ \


[trafficserver] 01/02: Fixed error getting h2 HEADERS frame after stream is closed

Posted by bc...@apache.org.
This is an automated email from the ASF dual-hosted git repository.

bcall pushed a commit to branch 8.0.x
in repository https://gitbox.apache.org/repos/asf/trafficserver.git

commit aaa9d0cff5293e022ee1213a7155c7192308a46c
Author: Bryan Call <bc...@apache.org>
AuthorDate: Mon Sep 10 09:59:47 2018 -0700

    Fixed error getting h2 HEADERS frame after stream is closed
    
    (cherry picked from commit 0d462f930d7f23ff7f43e0ecb63262d07b8d7c80)
---
 proxy/http2/Http2ConnectionState.cc | 5 +++++
 1 file changed, 5 insertions(+)

diff --git a/proxy/http2/Http2ConnectionState.cc b/proxy/http2/Http2ConnectionState.cc
index 01c0125..040a7e7 100644
--- a/proxy/http2/Http2ConnectionState.cc
+++ b/proxy/http2/Http2ConnectionState.cc
@@ -229,6 +229,11 @@ rcv_headers_frame(Http2ConnectionState &cstate, const Http2Frame &frame)
     }
   }
 
+  // Ignoring HEADERS frame on a closed stream.  The HdrHeap has gone away and it will core.
+  if (stream->get_state() == Http2StreamState::HTTP2_STREAM_STATE_CLOSED) {
+    return Http2Error(Http2ErrorClass::HTTP2_ERROR_CLASS_NONE);
+  }
+
   // keep track of how many bytes we get in the frame
   stream->request_header_length += payload_length;
   if (stream->request_header_length > Http2::max_request_header_size) {