Should use posix_error here.

This commit is contained in:
Hirokazu Yamamoto 2010-12-05 04:16:47 +00:00
parent 43b5a857d6
commit 26253bb09e
2 changed files with 4 additions and 6 deletions

View File

@ -248,7 +248,7 @@ class TestNtpath(unittest.TestCase):
self.assertFalse(ntpath.sameopenfile(tf1.fileno(), tf2.fileno()))
# Make sure invalid values don't cause issues on win32
if sys.platform == "win32":
with self.assertRaises(ValueError):
with self.assertRaises(OSError):
# Invalid file descriptors shouldn't display assert
# dialogs (#4804)
ntpath.sameopenfile(-1, -1)

View File

@ -2803,14 +2803,12 @@ posix__getfileinformation(PyObject *self, PyObject *args)
if (!PyArg_ParseTuple(args, "i:_getfileinformation", &fd))
return NULL;
if (!_PyVerify_fd(fd)) {
PyErr_SetString(PyExc_ValueError, "received invalid file descriptor");
return NULL;
}
if (!_PyVerify_fd(fd))
return posix_error();
hFile = (HANDLE)_get_osfhandle(fd);
if (hFile == INVALID_HANDLE_VALUE)
return win32_error("_getfileinformation", NULL);
return posix_error();
if (!GetFileInformationByHandle(hFile, &info))
return win32_error("_getfileinformation", NULL);