]> git.ipfire.org Git - thirdparty/haproxy.git/commit
BUG/MINOR: stream-int: don't try to read again when CF_READ_DONTWAIT is set
authorWilly Tarreau <w@1wt.eu>
Sat, 18 Nov 2017 10:26:20 +0000 (11:26 +0100)
committerWilly Tarreau <w@1wt.eu>
Mon, 20 Nov 2017 15:13:16 +0000 (16:13 +0100)
commit62dd698070c80d1ee4e779b1f7622386a45508dc
tree8e7631e9151aa382ae53194547f34d975899b9e2
parent33982cbdc062268d167a673274e3cc1cd7e3b232
BUG/MINOR: stream-int: don't try to read again when CF_READ_DONTWAIT is set

Commit 9aaf778 ("MAJOR: connection : Split struct connection into struct
connection and struct conn_stream.") had to change the way the stream
interface deals with incoming data to accomodate the mux. A break
statement got lost during a change, leading to the receive call being
performed twice even when CF_READ_DONTWAIT is set. The most noticeable
effect is that it made the bug described in commit 33982cb ("BUG/MAJOR:
stream: ensure analysers are always called upon close") much easier to
reproduce as it would appear even with an HTTP frontend.

Let's just restore the stream-interface flag and the break here, as in
the previous code.

No backport is needed as this was introduced during 1.8-dev.
src/stream_interface.c