]> git.ipfire.org Git - thirdparty/haproxy.git/commit
BUG/MEDIUM: channel: incorrect polling condition may delay event delivery
authorWilly Tarreau <w@1wt.eu>
Thu, 21 Apr 2016 10:12:45 +0000 (12:12 +0200)
committerWilly Tarreau <w@1wt.eu>
Thu, 21 Apr 2016 15:03:46 +0000 (17:03 +0200)
commit93dc478a04485272c723c0e510450d00b3c345c3
treec7d7f4ffdac404637414de0be3be93bf9d3c2edd
parent4b46a3e8cce58db03ee90718e39fb3bffb2b7273
BUG/MEDIUM: channel: incorrect polling condition may delay event delivery

The condition to poll for receive as implemented in channel_may_recv()
is still incorrect. If buf->o is null and buf->i is slightly larger than
chn->to_forward and at least as large as buf->size - maxrewrite, then
reading will be disabled. It may slightly delay some data delivery by
having first to forward pending bytes, but may also cause some random
issues with analysers that wait for some data before starting to forward
what they correctly parsed. For instance, a body analyser may be prevented
from seeing the data that only fits in the reserve.

This bug may also prevent an applet's chk_rcv() function from being called
when part of a buffer is released. It is possible (though not verified)
that this participated to some peers frozen session issues some people
have been facing.

This fix should be backported to 1.6 and 1.5 to ensure better coherency
with channel_recv_limit().
include/proto/channel.h