[ python-Bugs-629097 ] Race condition in asyncore poll

[SourceForge.net]

Bugs item #629097, was opened at 2002-10-26 10:48
Message generated for change (Comment added) made by akuchling
You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=105470&aid=629097&group_id=5470

Category: Python Library
Group: Python 2.3
>Status: Closed
>Resolution: Wont Fix
Priority: 5
Submitted By: Steve Alexander (stevea_zope)
Assigned to: A.M. Kuchling (akuchling)
Summary: Race condition in asyncore poll

Initial Comment:
In the following post to the Zope 3 developers' list, I
describe a race condition in the poll method of asyncore.

http://lists.zope.org/pipermail/zope3-dev/2002-October/003091.html

The problem is this:

There is a global dict socket_map. In the poll method,
socket_map is processed into appropriate arguments for
a select.select call. However, if a socket that is
represented socket_map is closed during the time
between the processing of socket_map and the
select.select call, that call will fail with a Bad File
Descriptor (EBADF) error.

One solution is to patch asyncore to catch EBADF errors
raised by select.select, and at that point see if the
file descriptors in the current version of socket_map 
are the same as in the processed data used for the
select.select call. If they are the same, re-raise the
error, otherwise, ignore the error.

In another email to the Zope 3 developers' list, Jeremy
Hylton queries whether there are any other similar
problems in asyncore.

http://lists.zope.org/pipermail/zope3-dev/2002-October/003093.html


----------------------------------------------------------------------

>Comment By: A.M. Kuchling (akuchling)
Date: 2004-07-10 13:15

Message:
Logged In: YES 
user_id=11375

Closing this bug.

----------------------------------------------------------------------

Comment By: Alexey Klimkin (klimkin)
Date: 2004-02-24 07:03

Message:
Logged In: YES 
user_id=410460

Yup, closing file in separate thread is program's (not
asyncore) error.

----------------------------------------------------------------------

Comment By: Guido van Rossum (gvanrossum)
Date: 2002-11-05 17:49

Message:
Logged In: YES 
user_id=6380

According to Jeremy, this is more a matter of "don't do that".

The right solution is to make sure that sockets are only
closed by the main thread (the thread running asyncore.loop()).

I wonder if we should just close this bug report?

----------------------------------------------------------------------

You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=105470&aid=629097&group_id=5470