commit 505989c0e701b105bb67b8b4c55681b0328171fe
author Benjamin Peterson <benjamin@python.org> Mon Aug 15 21:40:14 2016 -0700
committer Benjamin Peterson <benjamin@python.org> Mon Aug 15 21:40:14 2016 -0700
tree f931b4fce79fde8043222e9bf5f324fe4d4d107d
parent b97001ef46e599642affde61e31abace71486703

fail when negative values are passed to instr()

Modules/_cursesmodule.c

diff --git a/Modules/_cursesmodule.c b/Modules/_cursesmodule.c
index d0d7479..e478a57 100644
--- a/Modules/_cursesmodule.c
+++ b/Modules/_cursesmodule.c
@@ -1095,6 +1095,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,MIN(n,1023));
         break;
     case 2:
@@ -1105,6 +1109,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, MIN(n,1023));
         break;
     default: