[trafficserver] branch 8.0.x updated: HPACK: send back an error to the client when the index is invalid

[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


The following commit(s) were added to refs/heads/8.0.x by this push:
     new b0e4c5a  HPACK: send back an error to the client when the index is invalid
b0e4c5a is described below

commit b0e4c5ae2a85e7542a063923b1ad5be8769f3843
Author: Bryan Call <bc...@apache.org>
AuthorDate: Thu May 14 16:49:10 2020 -0700

    HPACK: send back an error to the client when the index is invalid
---
 proxy/http2/HPACK.cc | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

diff --git a/proxy/http2/HPACK.cc b/proxy/http2/HPACK.cc
index 1c45d1e..70a6805 100644
--- a/proxy/http2/HPACK.cc
+++ b/proxy/http2/HPACK.cc
@@ -796,7 +796,9 @@ decode_literal_header_field(MIMEFieldWrapper &header, const uint8_t *buf_start,
 
   // Decode header field name
   if (index) {
-    indexing_table.get_header_field(index, header);
+    if (indexing_table.get_header_field(index, header) == HPACK_ERROR_COMPRESSION_ERROR) {
+      return HPACK_ERROR_COMPRESSION_ERROR;
+    }
   } else {
     char *name_str        = nullptr;
     uint32_t name_str_len = 0;