You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@trafficserver.apache.org by ma...@apache.org on 2016/01/15 01:47:10 UTC
trafficserver git commit: TS-4118: CID 1347871: Add error handling in
HPACK_Encode
Repository: trafficserver
Updated Branches:
refs/heads/master 40500c710 -> 96b4d5201
TS-4118: CID 1347871: Add error handling in HPACK_Encode
Project: http://git-wip-us.apache.org/repos/asf/trafficserver/repo
Commit: http://git-wip-us.apache.org/repos/asf/trafficserver/commit/96b4d520
Tree: http://git-wip-us.apache.org/repos/asf/trafficserver/tree/96b4d520
Diff: http://git-wip-us.apache.org/repos/asf/trafficserver/diff/96b4d520
Branch: refs/heads/master
Commit: 96b4d52015549e211d1b2ee9d5c773f18f408153
Parents: 40500c7
Author: Masaori Koshiba <ma...@apache.org>
Authored: Tue Jan 12 10:07:56 2016 +0900
Committer: Masaori Koshiba <ma...@apache.org>
Committed: Thu Jan 14 14:21:20 2016 +0900
----------------------------------------------------------------------
proxy/http2/RegressionHPACK.cc | 5 +++++
1 file changed, 5 insertions(+)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/trafficserver/blob/96b4d520/proxy/http2/RegressionHPACK.cc
----------------------------------------------------------------------
diff --git a/proxy/http2/RegressionHPACK.cc b/proxy/http2/RegressionHPACK.cc
index e8826ff..f10b25e 100644
--- a/proxy/http2/RegressionHPACK.cc
+++ b/proxy/http2/RegressionHPACK.cc
@@ -423,6 +423,11 @@ REGRESSION_TEST(HPACK_Encode)(RegressionTest *t, int, int *pstatus)
int64_t len = http2_write_psuedo_headers(headers, buf, buf_len, indexing_table);
buf_len -= len;
+ if (len < 0) {
+ box.check(false, "http2_write_psuedo_headers returned negative value: %d", len);
+ break;
+ }
+
MIMEFieldIter field_iter;
bool cont = false;
len += http2_write_header_fragment(headers, field_iter, buf + len, buf_len, indexing_table, cont);