commit ef518155418409a2a7c8690a7438ee6322692ac6
author bnc <bnc@google.com> Thu Jul 18 05:36:44 2019 -0700
committer Copybara-Service <copybara-worker@google.com> Thu Jul 18 05:38:13 2019 -0700
tree 776b1aba45ebc9ac77dd87cc55c03ac85c501a38
parent 76992e7732e85ffcb24e288b436c68ce4a91a389

Stop processing data in QuicSpdyStream::OnDataAvailable() if connection is closed.

If the connection is closed, for example, QuicSpdyStream itself closes it
because a frame of invalid type is received, or because a QPACK error occurs,
then no more data should be fed into HttpDecoder.

gfe-relnote: n/a, change to QUIC v99-only class.
PiperOrigin-RevId: 258752154
Change-Id: Ibb60b00662160582a075e1a5353bab74e95490d5

quic/core/http/quic_spdy_stream.cc

diff --git a/quic/core/http/quic_spdy_stream.cc b/quic/core/http/quic_spdy_stream.cc
index b55e928..11cc233 100644
--- a/quic/core/http/quic_spdy_stream.cc
+++ b/quic/core/http/quic_spdy_stream.cc
@@ -661,7 +661,8 @@
   }
 
   iovec iov;
-  while (!reading_stopped() && decoder_.error() == QUIC_NO_ERROR) {
+  while (session()->connection()->connected() && !reading_stopped() &&
+         decoder_.error() == QUIC_NO_ERROR) {
     DCHECK_GE(sequencer_offset_, sequencer()->NumBytesConsumed());
     if (!sequencer()->PeekRegion(sequencer_offset_, &iov)) {
       break;