BUG/MINOR: mux-h2: Fix H2_CF_DEM_SHORT_READ value
authorChristopher Faulet <cfaulet@haproxy.com>
Wed, 10 Nov 2021 16:50:10 +0000 (17:50 +0100)
committerChristopher Faulet <cfaulet@haproxy.com>
Tue, 23 Nov 2021 15:34:07 +0000 (16:34 +0100)
The value for H2_CF_DEM_SHORT_READ flag is wrong. 2 bits are erroneously
set, 0x200 and 0x80000.  It is not an issue because both bits are not used
anywhere else.

The typo was introduced in the commit b5f7b5296 ("BUG/MEDIUM: mux-h2: Handle
remaining read0 cases on partial frames"). Thus this patch must also be
backported as far a 2.0.

(cherry picked from commit 47940c39e2195f03960a461081488c7603ad7167)
Signed-off-by: Christopher Faulet <cfaulet@haproxy.com>
(cherry picked from commit 484f10ad3bd735cd729b60b988a73e6f0c4dd391)
Signed-off-by: Christopher Faulet <cfaulet@haproxy.com>

src/mux_h2.c

index d7d7af7..1c1faef 100644 (file)
@@ -60,7 +60,7 @@ static const struct h2s *h2_idle_stream;
 #define H2_CF_DEM_BLOCK_ANY     0x000001F0  // aggregate of the demux flags above except DALLOC/DFULL
                                             // (SHORT_READ is also excluded)
 
-#define H2_CF_DEM_SHORT_READ    0x00080200  // demux blocked on incomplete frame
+#define H2_CF_DEM_SHORT_READ    0x00000200  // demux blocked on incomplete frame
 
 /* other flags */
 #define H2_CF_GOAWAY_SENT       0x00001000  // a GOAWAY frame was successfully sent