tree 93d26973611e1a2d83ba450a0ec36d1a6929f2ba
parent e70050e6c5642e04215b531ebf4aa0c8febb5959
author Willy Tarreau <w@1wt.eu> 1612523761 +0100
committer Christopher Faulet <cfaulet@haproxy.com> 1612535859 +0100

BUG/MEDIUM: mux-h2: do not quit the demux loop before setting END_REACHED

The demux loop could quit on missing data but the H2_CF_END_REACHED flag
would not be set in this case. This fixes a remaining situation where
previous commit f09612289 ("BUG/MEDIUM: mux-h2: handle remaining read0
cases") could not be sufficient and still leave CLOSE_WAIT. It's harder
to reproduce but was still observed in prod.

Now we quit via the end of the loop which already takes care of shutr.

This should be backported along with the patch above as far as 2.0.

(cherry picked from commit 133aaa9f110f5b78e57a23f9db0553e2978eca0e)
Signed-off-by: Willy Tarreau <w@1wt.eu>
(cherry picked from commit 02bd15b8db6990fd518544aa1a182afa5eaf7a2d)
Signed-off-by: Willy Tarreau <w@1wt.eu>
(cherry picked from commit a89aea1a1638591dab4aca8d1c2c71571957c74c)
Signed-off-by: Christopher Faulet <cfaulet@haproxy.com>
(cherry picked from commit 50d2e1bc7723aaf79e0c918565253d94dfe92c4a)
Signed-off-by: Christopher Faulet <cfaulet@haproxy.com>