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

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


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

commit 6691152bade08365011e79c515038615dc388d4a
Author: Bryan Call <bc...@apache.org>
AuthorDate: Thu May 14 16:46:13 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 2ebaaee..eefdef2 100644
--- a/proxy/http2/HPACK.cc
+++ b/proxy/http2/HPACK.cc
@@ -691,7 +691,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;
     uint64_t name_str_len = 0;

Reply via email to