commit 0cc85ef038daec284b1692eaf84f6ffdf91c3d92
author bnc <bnc@google.com> Tue Jul 14 07:25:03 2020 -0700
committer Copybara-Service <copybara-worker@google.com> Tue Jul 14 07:25:47 2020 -0700
tree 195f2766731d5d95056dc324ac97e00c7e77b2e7
parent a5812a0bf342960a3734c1db4e2b1372a0f59dc3

Fix hpack error code mapping to quicerrorcode.  not protected.

PiperOrigin-RevId: 321153785
Change-Id: I2e597fda8352763ccc80fda5bfd6f069483e85ba

quic/core/http/end_to_end_test.cc

diff --git a/quic/core/http/end_to_end_test.cc b/quic/core/http/end_to_end_test.cc
index a1c63c2..4973934 100644
--- a/quic/core/http/end_to_end_test.cc
+++ b/quic/core/http/end_to_end_test.cc
@@ -3797,7 +3797,7 @@
                 IsError(QUIC_QPACK_DECOMPRESSION_FAILED));
   } else {
     EXPECT_THAT(client_->connection_error(),
-                IsError(QUIC_HPACK_INDEX_VARINT_ERROR));
+                IsError(QUIC_HPACK_VALUE_TOO_LONG));
   }
 }
 

quic/core/http/quic_spdy_session.cc

diff --git a/quic/core/http/quic_spdy_session.cc b/quic/core/http/quic_spdy_session.cc
index f470322..b9acfef 100644
--- a/quic/core/http/quic_spdy_session.cc
+++ b/quic/core/http/quic_spdy_session.cc
@@ -139,21 +139,21 @@
     QuicErrorCode code;
     switch (error) {
       case Http2DecoderAdapter::SpdyFramerError::SPDY_HPACK_INDEX_VARINT_ERROR:
-        code = QUIC_HPACK_NAME_LENGTH_VARINT_ERROR;
+        code = QUIC_HPACK_INDEX_VARINT_ERROR;
         break;
       case Http2DecoderAdapter::SpdyFramerError::
           SPDY_HPACK_NAME_LENGTH_VARINT_ERROR:
-        code = QUIC_HPACK_VALUE_LENGTH_VARINT_ERROR;
+        code = QUIC_HPACK_NAME_LENGTH_VARINT_ERROR;
         break;
       case Http2DecoderAdapter::SpdyFramerError::
           SPDY_HPACK_VALUE_LENGTH_VARINT_ERROR:
-        code = QUIC_HPACK_NAME_TOO_LONG;
+        code = QUIC_HPACK_VALUE_LENGTH_VARINT_ERROR;
         break;
       case Http2DecoderAdapter::SpdyFramerError::SPDY_HPACK_NAME_TOO_LONG:
-        code = QUIC_HPACK_VALUE_TOO_LONG;
+        code = QUIC_HPACK_NAME_TOO_LONG;
         break;
       case Http2DecoderAdapter::SpdyFramerError::SPDY_HPACK_VALUE_TOO_LONG:
-        code = QUIC_HPACK_INDEX_VARINT_ERROR;
+        code = QUIC_HPACK_VALUE_TOO_LONG;
         break;
       case Http2DecoderAdapter::SpdyFramerError::SPDY_HPACK_NAME_HUFFMAN_ERROR:
         code = QUIC_HPACK_NAME_HUFFMAN_ERROR;