[Python-checkins] bpo-36814: ensure os.posix_spawn() handles None (GH-13144)
Victor Stinner
webhook-mailer at python.org
Thu May 9 22:00:11 EDT 2019
https://github.com/python/cpython/commit/948ed8c96b6912541a608591efe3e00fb520429a
commit: 948ed8c96b6912541a608591efe3e00fb520429a
branch: master
author: Anthony Shaw <anthony.p.shaw at gmail.com>
committer: Victor Stinner <vstinner at redhat.com>
date: 2019-05-10T04:00:06+02:00
summary:
bpo-36814: ensure os.posix_spawn() handles None (GH-13144)
Fix an issue where os.posix_spawn() would incorrectly raise a TypeError
when file_actions is None.
files:
A Misc/NEWS.d/next/Library/2019-05-06-23-13-26.bpo-36814.dSeMz_.rst
M Lib/test/test_posix.py
M Modules/posixmodule.c
diff --git a/Lib/test/test_posix.py b/Lib/test/test_posix.py
index 843402930ffc..5c93d0d507d2 100644
--- a/Lib/test/test_posix.py
+++ b/Lib/test/test_posix.py
@@ -1550,6 +1550,15 @@ def test_specify_environment(self):
with open(envfile) as f:
self.assertEqual(f.read(), 'bar')
+ def test_none_file_actions(self):
+ pid = self.spawn_func(
+ self.NOOP_PROGRAM[0],
+ self.NOOP_PROGRAM,
+ os.environ,
+ file_actions=None
+ )
+ self.assertEqual(os.waitpid(pid, 0), (pid, 0))
+
def test_empty_file_actions(self):
pid = self.spawn_func(
self.NOOP_PROGRAM[0],
diff --git a/Misc/NEWS.d/next/Library/2019-05-06-23-13-26.bpo-36814.dSeMz_.rst b/Misc/NEWS.d/next/Library/2019-05-06-23-13-26.bpo-36814.dSeMz_.rst
new file mode 100644
index 000000000000..3f40011b2650
--- /dev/null
+++ b/Misc/NEWS.d/next/Library/2019-05-06-23-13-26.bpo-36814.dSeMz_.rst
@@ -0,0 +1 @@
+Fix an issue where os.posix_spawnp() would incorrectly raise a TypeError when file_actions is None.
\ No newline at end of file
diff --git a/Modules/posixmodule.c b/Modules/posixmodule.c
index 221f7101b213..aa77094da06a 100644
--- a/Modules/posixmodule.c
+++ b/Modules/posixmodule.c
@@ -5465,7 +5465,7 @@ py_posix_spawn(int use_posix_spawnp, PyObject *module, path_t *path, PyObject *a
goto exit;
}
- if (file_actions != NULL) {
+ if (file_actions != NULL && file_actions != Py_None) {
/* There is a bug in old versions of glibc that makes some of the
* helper functions for manipulating file actions not copy the provided
* buffers. The problem is that posix_spawn_file_actions_addopen does not
More information about the Python-checkins
mailing list