From 7af65568ffd047de653ec9a87e4217bbfdc9d43c Mon Sep 17 00:00:00 2001 From: Benjamin Peterson Date: Mon, 29 Dec 2008 17:56:58 +0000 Subject: [PATCH] #4764 in io.open, set IOError.filename when trying to open a directory on POSIX platforms --- Lib/test/test_fileio.py | 1 + Misc/NEWS | 3 +++ Modules/_fileio.c | 8 ++++---- 3 files changed, 8 insertions(+), 4 deletions(-) diff --git a/Lib/test/test_fileio.py b/Lib/test/test_fileio.py index c9787795e33..d8cf415ed86 100644 --- a/Lib/test/test_fileio.py +++ b/Lib/test/test_fileio.py @@ -109,6 +109,7 @@ class AutoFileTests(unittest.TestCase): _fileio._FileIO('.', 'r') except IOError as e: self.assertNotEqual(e.errno, 0) + self.assertEqual(e.filename, ".") else: self.fail("Should have raised IOError") diff --git a/Misc/NEWS b/Misc/NEWS index c315af0f5e9..78d97808e48 100644 --- a/Misc/NEWS +++ b/Misc/NEWS @@ -12,6 +12,9 @@ What's New in Python 2.7 alpha 1 Core and Builtins ----------------- +- Issue #4764: With io.open, IOError.filename is set when trying to open a + directory on POSIX systems. + - Issue #4764: IOError.filename is set when trying to open a directory on POSIX systems. diff --git a/Modules/_fileio.c b/Modules/_fileio.c index 65cc99d45a8..ca12822e1db 100644 --- a/Modules/_fileio.c +++ b/Modules/_fileio.c @@ -98,7 +98,7 @@ fileio_new(PyTypeObject *type, PyObject *args, PyObject *kews) directories, so we need a check. */ static int -dircheck(PyFileIOObject* self) +dircheck(PyFileIOObject* self, char *name) { #if defined(HAVE_FSTAT) && defined(S_IFDIR) && defined(EISDIR) struct stat buf; @@ -109,8 +109,8 @@ dircheck(PyFileIOObject* self) PyObject *exc; internal_close(self); - exc = PyObject_CallFunction(PyExc_IOError, "(is)", - EISDIR, msg); + exc = PyObject_CallFunction(PyExc_IOError, "(iss)", + EISDIR, msg, name); PyErr_SetObject(PyExc_IOError, exc); Py_XDECREF(exc); return -1; @@ -271,7 +271,7 @@ fileio_init(PyObject *oself, PyObject *args, PyObject *kwds) #endif goto error; } - if(dircheck(self) < 0) + if(dircheck(self, name) < 0) goto error; }