From 013ec80ea89113af7ed46b6f30dedcba3c63d3e8 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?M=C3=A5rten=20Nordheim?= Date: Fri, 9 May 2025 11:31:48 +0200 Subject: [PATCH] Http2: check if stream was RSTed w/ bespoke function We have a function to check if a stream was locally reset - we should realistically only process HEADERS on RSTed streams for a limited amount of time. I forgot we had this mechanism in place already, let's update to use it for the recent fix. Amends 9224fdd3f4cccce7a8123ab398f6ff8745d759ce Pick-to: 6.9.1 6.8 Change-Id: I4826100704542f681341ceb84c4f85332f8ae03d Reviewed-by: Matthias Rauter (cherry picked from commit f95805670161b38da60602ada6128f03f7f9ceb2) Reviewed-by: Mate Barany Reviewed-by: Edward Welbourne --- src/network/access/qhttp2connection.cpp | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/network/access/qhttp2connection.cpp b/src/network/access/qhttp2connection.cpp index 090468c39a9..1e864041f62 100644 --- a/src/network/access/qhttp2connection.cpp +++ b/src/network/access/qhttp2connection.cpp @@ -1875,7 +1875,7 @@ void QHttp2Connection::handleContinuedHEADERS() const auto streamIt = m_streams.constFind(streamID); if (firstFrameType == FrameType::HEADERS) { - if (streamIt != m_streams.cend() && !streamIt.value()->wasReset()) { + if (streamIt != m_streams.cend() && !streamWasResetLocally(streamID)) { QHttp2Stream *stream = streamIt.value(); if (stream->state() != QHttp2Stream::State::HalfClosedLocal && stream->state() != QHttp2Stream::State::ReservedRemote