homepage

asyncore's module readwrite() function, used when invoking
asyncore.loop(use_poll=1), erroneously calls handle_read_event() when
receiving OOB (Out Of Band) data. handle_expt_event() should be called
instead.
The patch in attachment does that.
In addition I strongly think that POLLERR, POLLHUP and POLLNVAL events
handling is incorrect too.
As far as I read from here:
http://www.squarebox.co.uk/cgi-squarebox/manServer/usr/share/man/man0p/poll.h.0p
...they refers to the following events:
POLLERR 	An error has occurred (revents only).
POLLHUP 	Device has been disconnected ( revents only).
POLLNVAL	Invalid fd member (revents only).
They are actually associated to handle_expt_event() and this is
incorrect since it should be called only when receiving OOB data.
 if flags & (select.POLLERR | select.POLLHUP | select.POLLNVAL):
 obj.handle_expt_event()
I'm not sure what should be called instead, if handle_read_event or
handle_read or handle_error.
I tried to take a look at how Twisted manages the thing but it seems
that OOB is not even supported.
Maybe someone with more experience in using select.poll could clarify that.

While handle_expt() is documented has only being called when OOB data is 
known, it has become the catch-all for "something strange is happening 
on the socket".
The reason it wasn't changed/fixed in Python 2.6 is because a new method 
would need to be added, which would break previously existing asyncore-
derived applications.
I might still be able to fix it for 3.0 .

3.0 is a no-go, no non-documentation changes allowed.

This is a duplicate of #4501 which provides a better patch.

Setting #4501 as superseder.