From 5be666010e4df65dc4d831435cc92340ea369f94 Mon Sep 17 00:00:00 2001 From: Zackery Spytz Date: Fri, 23 Aug 2019 12:38:41 -0600 Subject: bpo-37549: os.dup() fails for standard streams on Windows 7 (GH-15389) --- Lib/test/test_os.py | 5 +++++ .../NEWS.d/next/Windows/2019-08-22-09-04-44.bpo-37549.TpKI3M.rst | 1 + Python/fileutils.c | 9 ++++++++- 3 files changed, 14 insertions(+), 1 deletion(-) create mode 100644 Misc/NEWS.d/next/Windows/2019-08-22-09-04-44.bpo-37549.TpKI3M.rst diff --git a/Lib/test/test_os.py b/Lib/test/test_os.py index 4285931..440cd6c 100644 --- a/Lib/test/test_os.py +++ b/Lib/test/test_os.py @@ -3476,6 +3476,11 @@ class FDInheritanceTests(unittest.TestCase): self.addCleanup(os.close, fd2) self.assertEqual(os.get_inheritable(fd2), False) + def test_dup_standard_stream(self): + fd = os.dup(1) + self.addCleanup(os.close, fd) + self.assertGreater(fd, 0) + @unittest.skipUnless(sys.platform == 'win32', 'win32-specific test') def test_dup_nul(self): # os.dup() was creating inheritable fds for character files. diff --git a/Misc/NEWS.d/next/Windows/2019-08-22-09-04-44.bpo-37549.TpKI3M.rst b/Misc/NEWS.d/next/Windows/2019-08-22-09-04-44.bpo-37549.TpKI3M.rst new file mode 100644 index 0000000..5345da8 --- /dev/null +++ b/Misc/NEWS.d/next/Windows/2019-08-22-09-04-44.bpo-37549.TpKI3M.rst @@ -0,0 +1 @@ +:func:`os.dup` no longer fails for standard streams on Windows 7. diff --git a/Python/fileutils.c b/Python/fileutils.c index 36a3c99..0c05424 100644 --- a/Python/fileutils.c +++ b/Python/fileutils.c @@ -1134,11 +1134,18 @@ set_inheritable(int fd, int inheritable, int raise, int *atomic_flag_works) flags = HANDLE_FLAG_INHERIT; else flags = 0; - if (!SetHandleInformation(handle, HANDLE_FLAG_INHERIT, flags)) { + + /* This check can be removed once support for Windows 7 ends. */ +#define CONSOLE_PSEUDOHANDLE(handle) (((ULONG_PTR)(handle) & 0x3) == 0x3 && \ + GetFileType(handle) == FILE_TYPE_CHAR) + + if (!CONSOLE_PSEUDOHANDLE(handle) && + !SetHandleInformation(handle, HANDLE_FLAG_INHERIT, flags)) { if (raise) PyErr_SetFromWindowsErr(0); return -1; } +#undef CONSOLE_PSEUDOHANDLE return 0; #else -- cgit v0.12