agent: Always emit a readable callback when pseudo-TCP data arrives
authorPhilip Withnall <philip.withnall@collabora.co.uk>
Thu, 9 Jan 2014 18:06:19 +0000 (18:06 +0000)
committerOlivier Crête <olivier.crete@collabora.com>
Fri, 31 Jan 2014 06:48:59 +0000 (01:48 -0500)
commitd285b5a53fe2446dc6778c334b30a965d08bfb56
treef33d8a8934af91d527b30ff85328d32564a296e5
parentcf9d3f18e6b0b0eaee701aaf61ae88b4819f360c
agent: Always emit a readable callback when pseudo-TCP data arrives

Previously, the pseudo-TCP implementation’s readable() callback would
only be invoked when new data was received if it was enabled. The
callback is enabled by reading all data from the TCP input buffer until
EWOULDBLOCK is returned.

Reading all that data is not possible if the client buffer is of a
limited size, and can lead to livelocks if the client reads exactly the
number of bytes in the TCP input buffer (i.e. its buffer size matches
the TCP buffer fill level).

Instead, always invoke the readable() callback. This might be slightly
less performant, but the whole pseudo-TCP implementation is a shambles
anyway, and the callbacks need removing, so why not?
agent/pseudotcp.c