[Python-checkins] cpython (3.4): fail when negative values are passed to instr()
benjamin.peterson
python-checkins at python.org
Tue Aug 16 00:44:19 EDT 2016
https://hg.python.org/cpython/rev/1de897915974
changeset: 102682:1de897915974
branch: 3.4
parent: 102655:dc1bacd79e66
user: Benjamin Peterson <benjamin at python.org>
date: Mon Aug 15 21:40:14 2016 -0700
summary:
fail when negative values are passed to instr()
files:
Lib/test/test_curses.py | 2 ++
Misc/NEWS | 4 ++--
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
--- a/Lib/test/test_curses.py
+++ b/Lib/test/test_curses.py
@@ -165,6 +165,8 @@
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):
diff --git a/Misc/NEWS b/Misc/NEWS
--- a/Misc/NEWS
+++ b/Misc/NEWS
@@ -13,8 +13,8 @@
Library
-------
-- 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
--- a/Modules/_cursesmodule.c
+++ b/Modules/_cursesmodule.c
@@ -1456,6 +1456,10 @@
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:
@@ -1466,6 +1470,10 @@
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:
--
Repository URL: https://hg.python.org/cpython
More information about the Python-checkins
mailing list