]> git.ipfire.org Git - thirdparty/haproxy.git/commit
BUG/MEDIUM: stream: Report write timeouts before testing the flags
authorChristopher Faulet <cfaulet@haproxy.com>
Wed, 12 Apr 2023 16:23:15 +0000 (18:23 +0200)
committerChristopher Faulet <cfaulet@haproxy.com>
Fri, 14 Apr 2023 10:13:09 +0000 (12:13 +0200)
commit915ba08b57b5dfc8d7c17edb6ff1714bc9411248
treede7cabd7ca87a4ed31293308fb109b62392fc604
parent925279ccf241200d219b84e3e00124d677925360
BUG/MEDIUM: stream: Report write timeouts before testing the flags

A regression was introduced when stream's timeouts were refactored. Write
timeouts are not testing is the right order. When timeous of the front SC
are handled, we must then test the read timeout on the request channel and
the write timeout on the response channel. But write timeout is tested on
the request channel instead. On the back SC, the same mix-up is performed.

We must be careful to handle timeouts before checking channel flags. To
avoid any confusions, all timeuts are handled first, on front and back SCs.
Then flags of the both channels are tested.

It is a 2.8-specific issue. No backport needed.
src/stream.c