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 <[email protected]> Authored: Tue Jan 12 10:07:56 2016 +0900 Committer: Masaori Koshiba <[email protected]> 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);
