[Python-checkins] cpython (merge 3.5 -> default): Issue #27066: Fixed SystemError if a custom opener (for open()) returns a
barry.warsaw
python-checkins at python.org
Wed Jun 8 17:57:35 EDT 2016
https://hg.python.org/cpython/rev/84c91d7d4667
changeset: 101812:84c91d7d4667
parent: 101810:09ec7f7322b5
parent: 101811:4af64ab34eef
user: Barry Warsaw <barry at python.org>
date: Wed Jun 08 17:54:43 2016 -0400
summary:
Issue #27066: Fixed SystemError if a custom opener (for open()) returns a
negative number without setting an exception.
files:
Lib/test/test_io.py | 16 ++++++++++++++++
Misc/NEWS | 3 +++
Modules/_io/fileio.c | 8 +++++++-
3 files changed, 26 insertions(+), 1 deletions(-)
diff --git a/Lib/test/test_io.py b/Lib/test/test_io.py
--- a/Lib/test/test_io.py
+++ b/Lib/test/test_io.py
@@ -809,6 +809,22 @@
with self.open("non-existent", "r", opener=opener) as f:
self.assertEqual(f.read(), "egg\n")
+ def test_bad_opener_negative_1(self):
+ # Issue #27066.
+ def badopener(fname, flags):
+ return -1
+ with self.assertRaises(ValueError) as cm:
+ open('non-existent', 'r', opener=badopener)
+ self.assertEqual(str(cm.exception), 'opener returned -1')
+
+ def test_bad_opener_other_negative(self):
+ # Issue #27066.
+ def badopener(fname, flags):
+ return -2
+ with self.assertRaises(ValueError) as cm:
+ open('non-existent', 'r', opener=badopener)
+ self.assertEqual(str(cm.exception), 'opener returned -2')
+
def test_fileio_closefd(self):
# Issue #4841
with self.open(__file__, 'rb') as f1, \
diff --git a/Misc/NEWS b/Misc/NEWS
--- a/Misc/NEWS
+++ b/Misc/NEWS
@@ -10,6 +10,9 @@
Core and Builtins
-----------------
+- Issue #27066: Fixed SystemError if a custom opener (for open()) returns a
+ negative number without setting an exception.
+
- Issue #26983: float() now always return an instance of exact float.
The deprecation warning is emitted if __float__ returns an instance of
a strict subclass of float. In a future versions of Python this can
diff --git a/Modules/_io/fileio.c b/Modules/_io/fileio.c
--- a/Modules/_io/fileio.c
+++ b/Modules/_io/fileio.c
@@ -420,7 +420,13 @@
self->fd = _PyLong_AsInt(fdobj);
Py_DECREF(fdobj);
- if (self->fd == -1) {
+ if (self->fd < 0) {
+ if (!PyErr_Occurred()) {
+ /* The opener returned a negative but didn't set an
+ exception. See issue #27066 */
+ PyErr_Format(PyExc_ValueError,
+ "opener returned %d", self->fd);
+ }
goto error;
}
}
--
Repository URL: https://hg.python.org/cpython
More information about the Python-checkins
mailing list