Issue #14829: Fix bisect and range() indexing with large indices (>= 2 ** 32) under 64-bit Windows.

pitrou · pitrou · commit b7d033db783b · 2012-05-16T14:39:36.000+02:00
(untested, because of Windows build issues under 3.x)
diff --git a/Misc/NEWS b/Misc/NEWS
@@ -34,6 +34,9 @@ Core and Builtins
 Library
 -------
 
+- Issue #14829: Fix bisect and range() indexing with large indices
+  (>= 2 ** 32) under 64-bit Windows.
+
 - Issue #14732: The _csv module now uses PEP 3121 module initialization.
   Patch by Robin Schreiber.
 
diff --git a/Modules/_bisectmodule.c b/Modules/_bisectmodule.c
@@ -3,6 +3,7 @@
 Converted to C by Dmitry Vasiliev (dima at hlabs.spb.ru).
 */
 
+#define PY_SSIZE_T_CLEAN
 #include "Python.h"
 
 static Py_ssize_t
@@ -195,8 +196,7 @@ insort_left(PyObject *self, PyObject *args, PyObject *kw)
             return NULL;
     } else {
         _Py_IDENTIFIER(insert);
-
-        result = _PyObject_CallMethodId(list, &PyId_insert, "iO", index, item);
+        result = _PyObject_CallMethodId(list, &PyId_insert, "nO", index, item);
         if (result == NULL)
             return NULL;
         Py_DECREF(result);
diff --git a/Objects/rangeobject.c b/Objects/rangeobject.c
@@ -308,7 +308,7 @@ compute_range_item(rangeobject *r, PyObject *arg)
 static PyObject *
 range_item(rangeobject *r, Py_ssize_t i)
 {
-    PyObject *res, *arg = PyLong_FromLong(i);
+    PyObject *res, *arg = PyLong_FromSsize_t(i);
     if (!arg) {
         return NULL;
     }

Original file line number	Diff line number	Diff line change
`@@ -308,7 +308,7 @@ compute_range_item(rangeobject r, PyObject arg)`
`308`	`308`	`static PyObject *`
`309`	`309`	`range_item(rangeobject *r, Py_ssize_t i)`
`310`	`310`	`{`
`311`		`- PyObject res, arg = PyLong_FromLong(i);`
	`311`	`+ PyObject res, arg = PyLong_FromSsize_t(i);`
`312`	`312`	`if (!arg) {`
`313`	`313`	`return NULL;`
`314`	`314`	`}`