Fix GOAWAY max frame length.
The stream id of a GOAWAY frame is delivered in varint 62 format. Thus we should allow maximum 8 bytes of frame payload.
gfe-relnote: protected by disabled v99 flag.
PiperOrigin-RevId: 274025730
Change-Id: I1685104558764eb04039bb8502d81495e890cdf1
diff --git a/quic/core/http/http_decoder.cc b/quic/core/http/http_decoder.cc
index 7fe994e..41632a9 100644
--- a/quic/core/http/http_decoder.cc
+++ b/quic/core/http/http_decoder.cc
@@ -8,6 +8,7 @@
#include "net/third_party/quiche/src/quic/core/http/http_frames.h"
#include "net/third_party/quiche/src/quic/core/quic_data_reader.h"
+#include "net/third_party/quiche/src/quic/core/quic_types.h"
#include "net/third_party/quiche/src/quic/platform/api/quic_bug_tracker.h"
#include "net/third_party/quiche/src/quic/platform/api/quic_fallthrough.h"
@@ -589,7 +590,7 @@
// This limit is arbitrary.
return 1024 * 1024;
case static_cast<uint64_t>(HttpFrameType::GOAWAY):
- return sizeof(QuicStreamId);
+ return VARIABLE_LENGTH_INTEGER_LENGTH_8;
case static_cast<uint64_t>(HttpFrameType::MAX_PUSH_ID):
return sizeof(PushId);
case static_cast<uint64_t>(HttpFrameType::DUPLICATE_PUSH):