diff options
| author | Benjamin Peterson <benjamin@python.org> | 2016-08-16 04:43:57 (GMT) |
|---|---|---|
| committer | Benjamin Peterson <benjamin@python.org> | 2016-08-16 04:43:57 (GMT) |
| commit | b1c6bdc76a1ac7bc40850f75dbc2dff0b361ee3d (patch) | |
| tree | bae0a3720e162ed790cccc6aada33923990cc908 | |
| parent | aa9560c6331c136cbe1e9676a2c48a1e0080a32a (diff) | |
| parent | 432ea4ff375f920fa4a469d34e5dad56fb33f915 (diff) | |
| download | cpython-b1c6bdc76a1ac7bc40850f75dbc2dff0b361ee3d.zip cpython-b1c6bdc76a1ac7bc40850f75dbc2dff0b361ee3d.tar.gz cpython-b1c6bdc76a1ac7bc40850f75dbc2dff0b361ee3d.tar.bz2 | |
merge 3.4
| -rw-r--r-- | Lib/test/test_curses.py | 2 | ||||
| -rw-r--r-- | Misc/NEWS | 4 | ||||
| -rw-r--r-- | Modules/_cursesmodule.c | 8 |
3 files changed, 12 insertions, 2 deletions
diff --git a/Lib/test/test_curses.py b/Lib/test/test_curses.py index 6bbb3fc..87c536f 100644 --- a/Lib/test/test_curses.py +++ b/Lib/test/test_curses.py @@ -190,6 +190,8 @@ class TestCurses(unittest.TestCase): self.assertRaises(ValueError, stdscr.getstr, -400) self.assertRaises(ValueError, stdscr.getstr, 2, 3, -400) + self.assertRaises(ValueError, stdscr.instr, -2) + self.assertRaises(ValueError, stdscr.instr, 2, 3, -2) def test_module_funcs(self): @@ -37,8 +37,8 @@ Library - Issue #26750: unittest.mock.create_autospec() now works properly for subclasses of property() and other data descriptors. -- In the curses module, raise an error if window.getstr() is passed a negative - value. +- In the curses module, raise an error if window.getstr() or window.instr() is + passed a negative value. - Issue #27760: Fix possible integer overflow in binascii.b2a_qp. diff --git a/Modules/_cursesmodule.c b/Modules/_cursesmodule.c index 06aa46c..960752c 100644 --- a/Modules/_cursesmodule.c +++ b/Modules/_cursesmodule.c @@ -1393,6 +1393,10 @@ PyCursesWindow_InStr(PyCursesWindowObject *self, PyObject *args) case 1: if (!PyArg_ParseTuple(args,"i;n", &n)) return NULL; + if (n < 0) { + PyErr_SetString(PyExc_ValueError, "'n' must be nonnegative"); + return NULL; + } rtn2 = winnstr(self->win, rtn, Py_MIN(n, 1023)); break; case 2: @@ -1403,6 +1407,10 @@ PyCursesWindow_InStr(PyCursesWindowObject *self, PyObject *args) case 3: if (!PyArg_ParseTuple(args, "iii;y,x,n", &y, &x, &n)) return NULL; + if (n < 0) { + PyErr_SetString(PyExc_ValueError, "'n' must be nonnegative"); + return NULL; + } rtn2 = mvwinnstr(self->win, y, x, rtn, Py_MIN(n,1023)); break; default: |