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

[zw...@apache.org]

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

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


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

commit b76477e2f2837beedee7769c9542a76856635710
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
    
    (cherry picked from commit 6691152bade08365011e79c515038615dc388d4a)
---
 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;