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

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


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

commit 62172e54da4be2c5f255826e5c7b3bda88786268
Author: Bryan Call <bc...@apache.org>
AuthorDate: Thu May 14 16:49:49 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 9ce3e5f..e06b7b7 100644
--- a/proxy/http2/HPACK.cc
+++ b/proxy/http2/HPACK.cc
@@ -791,7 +791,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;

Reply via email to