BUG/MEDIUM: h2/hpack: fix emission of HPACK DTSU after settings change
authorWilly Tarreau <w@1wt.eu>
Wed, 16 Feb 2022 13:28:14 +0000 (14:28 +0100)
committerWilly Tarreau <w@1wt.eu>
Wed, 16 Feb 2022 13:45:33 +0000 (14:45 +0100)
commita2a2e6db7247c19025270c8a5f48a747bf009f03
tree7f28eddae10bd8522199605adbbd5ad49ee233f9
parent413dbe777142e60b1d8cb24ae02f61279395602c
BUG/MEDIUM: h2/hpack: fix emission of HPACK DTSU after settings change

Sadly, despite particular care, commit 39a0a1e12 ("MEDIUM: h2/hpack: emit
a Dynamic Table Size Update after settings change") broke H2 when sending
DTSU. A missing negation on the flag caused the DTSU_EMITTED flag to be
lost and the DTSU to be sent again on the next stream, and possibly to
break flow control or a few other internal states.

This will have to be backported wherever the patch above was backported.

Thanks to Yves Lafon for notifying us with elements to reproduce the
issue!

(cherry picked from commit c7d85485a00bd9862ecb726ad1242c2ba724a8ca)
Signed-off-by: Willy Tarreau <w@1wt.eu>
src/mux_h2.c