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/14 00:02:04 UTC
[1/2] trafficserver git commit: TS-3513: HTTP/2 core dump - Fixed
problem with cursor not advancing during parsing
Repository: trafficserver
Updated Branches:
refs/heads/5.3.x 98a2e1bb3 -> 917db22c3
TS-3513: HTTP/2 core dump - Fixed problem with cursor not advancing during
parsing
Project: http://git-wip-us.apache.org/repos/asf/trafficserver/repo
Commit: http://git-wip-us.apache.org/repos/asf/trafficserver/commit/661297df
Tree: http://git-wip-us.apache.org/repos/asf/trafficserver/tree/661297df
Diff: http://git-wip-us.apache.org/repos/asf/trafficserver/diff/661297df
Branch: refs/heads/5.3.x
Commit: 661297dfae4d3a592a39249b74ef57e748b10805
Parents: 98a2e1b
Author: Bryan Call <bc...@apache.org>
Authored: Mon Apr 13 16:56:35 2015 -0500
Committer: Bryan Call <bc...@apache.org>
Committed: Mon Apr 13 16:56:35 2015 -0500
----------------------------------------------------------------------
proxy/http2/HPACK.cc | 2 +-
proxy/http2/HTTP2.cc | 6 +++---
2 files changed, 4 insertions(+), 4 deletions(-)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/trafficserver/blob/661297df/proxy/http2/HPACK.cc
----------------------------------------------------------------------
diff --git a/proxy/http2/HPACK.cc b/proxy/http2/HPACK.cc
index f12d7e4..4dd4eb0 100644
--- a/proxy/http2/HPACK.cc
+++ b/proxy/http2/HPACK.cc
@@ -534,7 +534,7 @@ decode_string(Arena &arena, char **str, uint32_t &str_length, const uint8_t *buf
return HPACK_ERROR_COMPRESSION_ERROR;
p += len;
- if (encoded_string_len > HEADER_FIELD_LIMIT_LENGTH || buf_start + encoded_string_len >= buf_end) {
+ if (encoded_string_len > HEADER_FIELD_LIMIT_LENGTH || (p + encoded_string_len) > buf_end) {
return HPACK_ERROR_COMPRESSION_ERROR;
}
http://git-wip-us.apache.org/repos/asf/trafficserver/blob/661297df/proxy/http2/HTTP2.cc
----------------------------------------------------------------------
diff --git a/proxy/http2/HTTP2.cc b/proxy/http2/HTTP2.cc
index f971938..6a8b290 100644
--- a/proxy/http2/HTTP2.cc
+++ b/proxy/http2/HTTP2.cc
@@ -629,10 +629,10 @@ http2_write_header_fragment(HTTPHdr *in, MIMEFieldIter &field_iter, uint8_t *out
int64_t
http2_parse_header_fragment(HTTPHdr *hdr, IOVec iov, Http2DynamicTable &dynamic_table, bool cont)
{
- uint8_t *buf_start = (uint8_t *)iov.iov_base;
- uint8_t *buf_end = (uint8_t *)iov.iov_base + iov.iov_len;
+ const uint8_t *buf_start = (uint8_t *)iov.iov_base;
+ const uint8_t *buf_end = buf_start + iov.iov_len;
- uint8_t *cursor = buf_start;
+ uint8_t *cursor = (uint8_t *)iov.iov_base; // place the cursor at the start
HdrHeap *heap = hdr->m_heap;
HTTPHdrImpl *hh = hdr->m_http;
[2/2] trafficserver git commit: TS-3513: Updated CHANGES
Posted by bc...@apache.org.
TS-3513: Updated CHANGES
Project: http://git-wip-us.apache.org/repos/asf/trafficserver/repo
Commit: http://git-wip-us.apache.org/repos/asf/trafficserver/commit/917db22c
Tree: http://git-wip-us.apache.org/repos/asf/trafficserver/tree/917db22c
Diff: http://git-wip-us.apache.org/repos/asf/trafficserver/diff/917db22c
Branch: refs/heads/5.3.x
Commit: 917db22c33fcdb41478e9c24ba0c09a219206ff8
Parents: 661297d
Author: Bryan Call <bc...@apache.org>
Authored: Mon Apr 13 17:01:50 2015 -0500
Committer: Bryan Call <bc...@apache.org>
Committed: Mon Apr 13 17:01:50 2015 -0500
----------------------------------------------------------------------
CHANGES | 3 +++
1 file changed, 3 insertions(+)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/trafficserver/blob/917db22c/CHANGES
----------------------------------------------------------------------
diff --git a/CHANGES b/CHANGES
index 41e839b..9fdc677 100644
--- a/CHANGES
+++ b/CHANGES
@@ -1,6 +1,9 @@
-*- coding: utf-8 -*-
Changes with Apache Traffic Server 5.3.0
+ *) [TS-3513] HTTP/2 core dump. Fixed problem with cursor not advancing
+ during parsing
+
*) [TS-3514] Atomic check for gcc >4.1 breaks with gcc 5.0
Author: Marcin Juszkiewicz <mj...@redhat.com>