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 Hisham Muhammad
Recipients Hisham Muhammad
Date 2015-03-02.16:50:12
SpamBayes Score -1.0
Marked as misclassified Yes
Message-id <1425315012.54.0.714244886784.issue23564@psf.upfronthosting.co.za>
In-reply-to
Content
In Modules/_posixsubprocess.c, (the helper module for Lib/subprocess.py) there's a function called _sanity_check_python_fd_sequence which checks, as its comment says, if the fds in the sequence are all positive and sorted.

The check to verify if it is sorted is incorrect (missing the update to the prev_fd variable), and also it is missing a test to see if fd's are repeated (which they shouldn't be, so the test should use <= rather than <). 

The attached patch, written against Python 3.4.3 source code, fixes it.
History
Date User Action Args
2015-03-02 16:50:12Hisham Muhammadsetrecipients: + Hisham Muhammad
2015-03-02 16:50:12Hisham Muhammadsetmessageid: <1425315012.54.0.714244886784.issue23564@psf.upfronthosting.co.za>
2015-03-02 16:50:12Hisham Muhammadlinkissue23564 messages
2015-03-02 16:50:12Hisham Muhammadcreate