This issue tracker has been migrated to GitHub, and is currently read-only.
For more information, see the GitHub FAQs in the Python's Developer Guide.

Author eryksun
Recipients Vyacheslav Grigoryev, eryksun, paul.moore, steve.dower, tim.golden, zach.ware
Date 2016-10-18.05:32:04
SpamBayes Score -1.0
Marked as misclassified Yes
Message-id <1476768724.88.0.580433143493.issue28462@psf.upfronthosting.co.za>
In-reply-to
Content
Due to a race condition, the Popen call in the second process is inadvertently inheriting the handle for the write end of the pipe that's created for the first process. Thus stdout.readline() in the first thread doesn't see EOF until that handle is closed. 

For the 2nd process, since you don't need to inherit standard handles, you can pass close_fds=True. In general if you do need to inherit standard handles in processes that are created concurrently, you can synchronize on a lock to ensure Popen properly closes inheritable handles. For example:

    import threading
    import subprocess

    class Popen(subprocess.Popen):
        _execute_lock = threading.Lock()
        def __init__(self, *args, **kwds):
            with self._execute_lock:
                super(Popen, self).__init__(*args, **kwds)

In Python 3 this should be addressed by implementing the suggestion in issue 19764 to use PROC_THREAD_ATTRIBUTE_HANDLE_LIST.
History
Date User Action Args
2016-10-18 05:32:04eryksunsetrecipients: + eryksun, paul.moore, tim.golden, zach.ware, steve.dower, Vyacheslav Grigoryev
2016-10-18 05:32:04eryksunsetmessageid: <1476768724.88.0.580433143493.issue28462@psf.upfronthosting.co.za>
2016-10-18 05:32:04eryksunlinkissue28462 messages
2016-10-18 05:32:04eryksuncreate