Commit message (Collapse) | Author | Age | Files | Lines | |
---|---|---|---|---|---|
* | Issue #25764: Merge subprocess fix from 3.4 into 3.5 | Martin Panter | 2015-12-05 | 1 | -17/+18 |
|\ | |||||
| * | Issue #25764: Preserve subprocess fork exception when preexec_fn used | Martin Panter | 2015-11-30 | 1 | -17/+18 |
| | | | | | | | | Also fix handling of failure to release the import lock. | ||||
* | | Fixes #23564: Fix a partially broken sanity check in the _posixsubprocess | Gregory P. Smith | 2015-11-16 | 1 | -1/+2 |
| | | | | | | | | | | internals regarding how fds_to_pass were passed to the child. The bug had no actual impact as subprocess.py already avoided it. | ||||
* | | Close #24784: Fix compilation without thread support | Victor Stinner | 2015-10-11 | 1 | -2/+10 |
| | | | | | | | | | | | | | | | | Add "#ifdef WITH_THREAD" around cals to: * PyGILState_Check() * _PyImport_AcquireLock() * _PyImport_ReleaseLock() | ||||
* | | Fix computation of max_fd on OpenBSD. Issue #23852. | Gregory P. Smith | 2015-04-26 | 1 | -0/+10 |
|\ \ | |/ | |||||
| * | Fix computation of max_fd on OpenBSD. Issue #23852. | Gregory P. Smith | 2015-04-26 | 1 | -0/+10 |
| | | |||||
* | | Issue9951: update _hashopenssl and md5module to use _Py_strhex(). | Gregory P. Smith | 2015-04-25 | 1 | -1/+1 |
| | | | | | | | | Also update _posixsubprocess to use Py_hexdigits instead of its own constant. | ||||
* | | Merge 3.4 (_posixsubprocess) | Victor Stinner | 2015-04-02 | 1 | -6/+6 |
|\ \ | |/ | |||||
| * | Issue #23851: close() must not be retried when it fails with EINTR | Victor Stinner | 2015-04-02 | 1 | -6/+6 |
| | | | | | | | | See the PEP 475 for the rationale. | ||||
* | | Issue #23836: Use _Py_write_noraise() to retry on EINTR in child_exec() of | Victor Stinner | 2015-04-01 | 1 | -9/+10 |
| | | | | | | | | _posixsubprocess | ||||
* | | Issue #23694: Fix usage of _Py_open() in the _posixsubprocess module | Victor Stinner | 2015-03-30 | 1 | -2/+1 |
| | | | | | | | | | | | | | | Don't call _Py_open() from _close_open_fds_safe() because it is call just after fork(). It's not good to play with locks (the GIL) between fork() and exec(). Use instead _Py_open_noraise() which doesn't touch to the GIL. | ||||
* | | Issue #23694: Enhance _Py_open(), it now raises exceptions | Victor Stinner | 2015-03-17 | 1 | -0/+1 |
|/ | | | | | | | | * _Py_open() now raises exceptions on error. If open() fails, it raises an OSError with the filename. * _Py_open() now releases the GIL while calling open() * Add _Py_open_noraise() when _Py_open() cannot be used because the GIL is not held | ||||
* | Issue #22290: Fix error handling in the _posixsubprocess module. | Victor Stinner | 2014-10-05 | 1 | -6/+14 |
| | | | | | | | | * Don't call the garbage collector with an exception set: it causes an assertion to fail in debug mode. * Enhance also error handling if allocating an array for the executable list failed. * Add an unit test for 4 different errors in the _posixsubprocess module. | ||||
* | Fix the comment to not refer to the removed end_fd parameter. | Gregory P. Smith | 2014-06-01 | 1 | -2/+2 |
| | |||||
* | Don't restrict ourselves to a "max" fd when closing fds before exec() | Gregory P. Smith | 2014-06-01 | 1 | -43/+44 |
| | | | | | | | | | when we have a way to get an actual list of all open fds from the OS. Fixes issue #21618: The subprocess module would ignore fds that were inherited by the calling process and already higher than POSIX resource limits would otherwise allow. On systems with a functioning /proc/self/fd or /dev/fd interface the max is now ignored and all fds are closed. | ||||
* | Add conditional code for android's lack of definition of SYS_getdent64. | Gregory P. Smith | 2014-04-14 | 1 | -0/+6 |
| | | | | | | Fixes issue20307. No Misc/NEWS entry because frankly this is an esoteric platform for anyone to be figuring out how to cross compile CPython for. | ||||
* | Undo supposed fix for Issue #15798 until I understand why this is | Gregory P. Smith | 2013-12-01 | 1 | -1/+5 |
|\ | | | | | | | | | causing test_multiprocessing_forkserver and test_multiprocessing_spawn failures on head (3.4). | ||||
| * | Undo supposed fix for Issue #15798 until I understand why this is | Gregory P. Smith | 2013-12-01 | 1 | -1/+5 |
| | | | | | | | | | | causing test_multiprocessing_forkserver and test_multiprocessing_spawn failures on head (3.4). | ||||
* | | Fixes Issue #15798 - subprocess.Popen() no longer fails if file | Gregory P. Smith | 2013-12-01 | 1 | -5/+1 |
|\ \ | |/ | | | | | descriptor 0, 1 or 2 is closed. | ||||
| * | Fixes Issue #15798 - subprocess.Popen() no longer fails if file | Gregory P. Smith | 2013-12-01 | 1 | -5/+1 |
| | | | | | | | | descriptor 0, 1 or 2 is closed. | ||||
* | | Issue #18571: Implementation of the PEP 446: file descriptors and file handles | Victor Stinner | 2013-08-27 | 1 | -89/+51 |
| | | | | | | | | | | are now created non-inheritable; add functions os.get/set_inheritable(), os.get/set_handle_inheritable() and socket.socket.get/set_inheritable(). | ||||
* | | Merge. | Charles-François Natali | 2013-08-25 | 1 | -11/+12 |
|\ \ | |/ | |||||
| * | Issue #18763: subprocess: The file descriptors are now closed after calling the | Charles-François Natali | 2013-08-25 | 1 | -11/+12 |
| | | | | | | | | preexec_fn callback, which may open file descriptors. | ||||
* | | Fixes Issue #16962: Use getdents64 instead of the obsolete getdents syscall | Gregory P. Smith | 2013-03-03 | 1 | -14/+8 |
|\ \ | |/ | | | | | in the subprocess module on Linux. | ||||
| * | Issue #16962: Use getdents64 instead of the obsolete getdents syscall in | Gregory P. Smith | 2013-03-03 | 1 | -14/+8 |
| | | | | | | | | the subprocess module on Linux. | ||||
* | | Raise our own SubprocessError rather than a RuntimeError in when dealing with | Gregory P. Smith | 2012-11-11 | 1 | -1/+1 |
|/ | | | | odd rare errors coming from the subprocess module. | ||||
* | Fixes Issue #16114: The subprocess module no longer provides a | Gregory P. Smith | 2012-10-10 | 1 | -1/+6 |
|\ | | | | | | | | | | | misleading error message stating that args[0] did not exist when either the cwd or executable keyword arguments specified a path that did not exist. | ||||
| * | Fixes Issue #16114: The subprocess module no longer provides a | Gregory P. Smith | 2012-10-10 | 1 | -1/+6 |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | misleading error message stating that args[0] did not exist when either the cwd or executable keyword arguments specified a path that did not exist. It now keeps track of if the child got as far as preexec and reports it if not back to the parent via a special "noexec" error message value in the error pipe so that the cwd can be blamed for a failed chdir instead of the exec of the executable being blamed instead. The executable is also always reported accurately when exec fails. Unittests enhanced to cover these cases. | ||||
* | | Merge with 3.2 | Ross Lagerwall | 2012-08-24 | 1 | -1/+3 |
|\ \ | |/ | |||||
| * | Issue 15777: Fix a refleak in _posixsubprocess. | Ross Lagerwall | 2012-08-24 | 1 | -1/+3 |
| | | | | | | | | It was exposed by 03c98d05b140 and dbbf3ccf72e8. | ||||
* | | Merge 3.2. | Stefan Krah | 2012-08-20 | 1 | -0/+2 |
|\ \ | |/ | |||||
| * | Issue #15738: Fix a missing NULL check in subprocess_fork_exec(). | Stefan Krah | 2012-08-20 | 1 | -0/+2 |
| | | |||||
* | | Issue #15604: Update uses of PyObject_IsTrue() to check for and handle ↵ | Antoine Pitrou | 2012-08-15 | 1 | -4/+3 |
|\ \ | |/ | | | | | | | | | errors correctly. Patch by Serhiy Storchaka. | ||||
| * | Issue #15604: Update uses of PyObject_IsTrue() to check for and handle ↵ | Antoine Pitrou | 2012-08-15 | 1 | -0/+2 |
| | | | | | | | | | | | | errors correctly. Patch by Serhiy Storchaka. | ||||
* | | Fixes issue #15000: support the odd x32 abi on posixsubprocess's system call. | Gregory P. Smith | 2012-06-05 | 1 | -0/+7 |
|\ \ | |/ | |||||
| * | Fixes issue #15000: support the odd x32 abi on posixsubprocess's system call. | Gregory P. Smith | 2012-06-05 | 1 | -0/+7 |
| | | |||||
| * | Issue #14359: Only use O_CLOEXEC in _posixmodule.c if it is defined. | Ross Lagerwall | 2012-03-19 | 1 | -1/+12 |
| | | | | | | | | Based on patch from Hervé Coatanhay. | ||||
* | | Issue 14359: Only use O_CLOEXEC in _posixmodule.c if it is defined. | Ross Lagerwall | 2012-03-18 | 1 | -1/+12 |
| | | | | | | | | Based on patch from Hervé Coatanhay. | ||||
* | | Use ANSI C prototype instead of K&R style. | Ross Lagerwall | 2012-03-07 | 1 | -1/+1 |
| | | |||||
* | | merge 3.2 | Benjamin Peterson | 2012-01-23 | 1 | -24/+31 |
|\ \ | |/ | |||||
| * | fix declaration style | Benjamin Peterson | 2012-01-23 | 1 | -24/+31 |
| | | |||||
* | | Fix FreeBSD, NetBSD and OpenBSD behavior of the issue #8052 fix. | Gregory P. Smith | 2012-01-22 | 1 | -9/+47 |
|\ \ | |/ | |||||
| * | Fix FreeBSD, NetBSD and OpenBSD behavior of the issue #8052 fix. | Gregory P. Smith | 2012-01-22 | 1 | -9/+47 |
| | | |||||
* | | bugfix for *BSD platforms. (oops messed up #define) | Gregory P. Smith | 2012-01-21 | 1 | -4/+2 |
|\ \ | |/ | |||||
| * | Another issue #8052 bugfix (related to previous commit). | Gregory P. Smith | 2012-01-21 | 1 | -4/+2 |
| | | | | | | | | "oops" while rearranging the #defines. | ||||
* | | bugfix for issue 8052 fixes on *BSD platforms. | Gregory P. Smith | 2012-01-21 | 1 | -9/+13 |
|\ \ | |/ | |||||
| * | Bugfix for issue #8052 fix on *BSD variants. | Gregory P. Smith | 2012-01-21 | 1 | -9/+13 |
| | | | | | | | | | | Many lack readdir64, use readdir. Only use readdir64 on solaris where it is required to work around a solaris bug. | ||||
* | | Fixes issue #8052: The posix subprocess module's close_fds behavior was | Gregory P. Smith | 2012-01-21 | 1 | -35/+258 |
|\ \ | |/ | | | | | | | | | | | | | suboptimal by closing all possible file descriptors rather than just the open ones in the child process before exec(). It now closes only the open fds when it is possible to safely determine what those are. | ||||
| * | Fixes issue #8052: The posix subprocess module's close_fds behavior was | Gregory P. Smith | 2012-01-21 | 1 | -35/+258 |
| | | | | | | | | | | | | | | | | suboptimal by closing all possible file descriptors rather than just the open ones in the child process before exec(). It now closes only the open fds when it is possible to safely determine what those are. | ||||
* | | Avoid the compiler warning about the unused return value. | Gregory P. Smith | 2012-01-21 | 1 | -6/+9 |
|\ \ | |/ |