diff options
| author | Steve Dower <steve.dower@microsoft.com> | 2015-03-14 11:39:18 -0700 |
|---|---|---|
| committer | Steve Dower <steve.dower@microsoft.com> | 2015-03-14 11:39:18 -0700 |
| commit | ba26e4dedbd1f2c4982d3dc79268f6ef8b6e1e9e (patch) | |
| tree | ebaca844679d46f2dd98d2d591841f848b8e35af /Python/fileutils.c | |
| parent | 2a7f8e0f24b10d8159baa93ae92c388014e62e01 (diff) | |
| parent | 92ce4bab13699db0b1e8551c111b1bc1dae84192 (diff) | |
| download | cpython-ba26e4dedbd1f2c4982d3dc79268f6ef8b6e1e9e.tar.gz | |
Fixes incorrect use of GetLastError where errno should be used.
Diffstat (limited to 'Python/fileutils.c')
| -rw-r--r-- | Python/fileutils.c | 12 |
1 files changed, 6 insertions, 6 deletions
diff --git a/Python/fileutils.c b/Python/fileutils.c index 6502823535..c6cdb19fbe 100644 --- a/Python/fileutils.c +++ b/Python/fileutils.c @@ -733,7 +733,7 @@ get_inheritable(int fd, int raise) handle = (HANDLE)_get_osfhandle(fd); if (handle == INVALID_HANDLE_VALUE) { if (raise) - PyErr_SetFromWindowsErr(0); + PyErr_SetFromErrno(PyExc_OSError); return -1; } @@ -788,10 +788,10 @@ set_inheritable(int fd, int inheritable, int raise, int *atomic_flag_works) if (atomic_flag_works != NULL && !inheritable) { if (*atomic_flag_works == -1) { - int inheritable = get_inheritable(fd, raise); - if (inheritable == -1) + int isInheritable = get_inheritable(fd, raise); + if (isInheritable == -1) return -1; - *atomic_flag_works = !inheritable; + *atomic_flag_works = !isInheritable; } if (*atomic_flag_works) @@ -808,7 +808,7 @@ set_inheritable(int fd, int inheritable, int raise, int *atomic_flag_works) handle = (HANDLE)_get_osfhandle(fd); if (handle == INVALID_HANDLE_VALUE) { if (raise) - PyErr_SetFromWindowsErr(0); + PyErr_SetFromErrno(PyExc_OSError); return -1; } @@ -1167,7 +1167,7 @@ _Py_dup(int fd) #ifdef MS_WINDOWS handle = (HANDLE)_get_osfhandle(fd); if (handle == INVALID_HANDLE_VALUE) { - PyErr_SetFromWindowsErr(0); + PyErr_SetFromErrno(PyExc_OSError); return -1; } |