Fix poll() if POLLIN == ROLLRDNORM|POLLRDBAND
authorDaphne Pfister <daphnediane@mac.com>
Sun, 8 Sep 2013 20:25:11 +0000 (16:25 -0400)
committerUli Schlachter <psychon@znc.in>
Sun, 8 Sep 2013 21:01:11 +0000 (23:01 +0200)
commitf1405d9fe4a6ddcae24585ba254389a4c4f4c8c9
treed51f0043513e6f39c570c20ae116da9674d40309
parentac47e0ecdb46aa91b191a59364437a8f65947467
Fix poll() if POLLIN == ROLLRDNORM|POLLRDBAND

It seems like POLLIN is specified as equivalent to POLLRDNORM | POLLRDBAND. Some
systems (e.g. QNX and HP-UX) take this literaly and have POLLIN defined as the
above bit combination. Other systems (e.g. Linux) have POLLIN as just a single
bit.

This means that if no out-of-band data is available (which should never be the
case), the result of poll() will not fulfil (fd.revents & POLLIN) == POLLIN on
QNX, because the POLLRDBAND bit is not set.

In other words, even though poll() signaled that the fd is readable, xcb would
not read from the file descriptor.

Fix this by checking if any bits from POLLIN are set in the result of poll(),
instead of all of them.

(This change was independently done by seanb@qnx.com as well)

Fixes: https://bugs.freedesktop.org/show_bug.cgi?id=38001
Acked-by: Julien Cristau <jcristau@debian.org>
Signed-off-by: Uli Schlachter <psychon@znc.in>
src/xcb_conn.c