commit 97b690b069ce413a9e8d36aba12d868957ef9151
author ianswett <ianswett@google.com> Thu May 02 15:12:43 2019 -0700
committer Copybara-Service <copybara-worker@google.com> Thu May 02 15:13:18 2019 -0700
tree 10840d0b01d33a7bfcfca1656675cbfc34a8923a
parent 994df05d82ce89d04375e4a985bc3901ceb0627c

gfe-relnote: Close the connection if a STOP_WAITING frame is received in v44 or above.  Protected by gfe2_reloadable_flag_quic_do_not_accept_stop_waiting.

PiperOrigin-RevId: 246399630
Change-Id: I9a810abb4d409f114f3e565ae6c5a37551073f6c

quic/core/quic_framer_test.cc

diff --git a/quic/core/quic_framer_test.cc b/quic/core/quic_framer_test.cc
index a717168..3c0b66d 100644
--- a/quic/core/quic_framer_test.cc
+++ b/quic/core/quic_framer_test.cc
@@ -4097,6 +4097,15 @@
 
   std::unique_ptr<QuicEncryptedPacket> encrypted(
       AssemblePacketFromFragments(fragments));
+  if (GetQuicReloadableFlag(quic_do_not_accept_stop_waiting) &&
+      version_.transport_version >= QUIC_VERSION_44) {
+    EXPECT_FALSE(framer_.ProcessPacket(*encrypted));
+    EXPECT_EQ(QUIC_INVALID_STOP_WAITING_DATA, framer_.error());
+    EXPECT_EQ("STOP WAITING not supported in version 44+.",
+              framer_.detailed_error());
+    return;
+  }
+
   EXPECT_TRUE(framer_.ProcessPacket(*encrypted));
 
   EXPECT_EQ(QUIC_NO_ERROR, framer_.error());
@@ -4114,7 +4123,9 @@
 }
 
 TEST_P(QuicFramerTest, InvalidNewStopWaitingFrame) {
-  if (version_.transport_version == QUIC_VERSION_99) {
+  if (version_.transport_version == QUIC_VERSION_99 ||
+      (GetQuicReloadableFlag(quic_do_not_accept_stop_waiting) &&
+       version_.transport_version >= QUIC_VERSION_44)) {
     return;
   }
   SetDecrypterLevel(ENCRYPTION_FORWARD_SECURE);