Index: Python/getargs.c =================================================================== --- Python/getargs.c (revision 61662) +++ Python/getargs.c (working copy) @@ -663,7 +663,6 @@ } case 'n': /* Py_ssize_t */ -#if SIZEOF_SIZE_T != SIZEOF_LONG { PyObject *iobj; Py_ssize_t *p = va_arg(*p_va, Py_ssize_t *); @@ -672,14 +671,13 @@ return converterr("integer", arg, msgbuf, bufsize); iobj = PyNumber_Index(arg); if (iobj != NULL) - ival = PyLong_AsSsize_t(arg); + ival = PyLong_AsSsize_t(iobj); if (ival == -1 && PyErr_Occurred()) return converterr("integer", arg, msgbuf, bufsize); *p = ival; break; } -#endif - /* Fall through from 'n' to 'l' if Py_ssize_t is int */ + case 'l': {/* long int */ long *p = va_arg(*p_va, long *); long ival; Index: Objects/abstract.c =================================================================== --- Objects/abstract.c (revision 61662) +++ Objects/abstract.c (working copy) @@ -1220,6 +1220,7 @@ PyObject * PyNumber_Index(PyObject *item) { + PyNumberMethods *m; PyObject *result = NULL; if (item == NULL) return null_error(); @@ -1238,7 +1239,18 @@ return NULL; } } - else { + m = item->ob_type->tp_as_number; + else if (m && m->nb_int != NULL) { + result = item->ob_type->tp_as_number->nb_int(item); + if (result && !PyLong_Check(result)) { + PyErr_Format(PyExc_TypeError, + "__int__ returned non-int " + "(type %.200s)", + result->ob_type->tp_name); + Py_DECREF(result); + return NULL; + } + } else { PyErr_Format(PyExc_TypeError, "'%.200s' object cannot be interpreted " "as an integer", item->ob_type->tp_name); Index: Lib/test/test_getargs2.py =================================================================== --- Lib/test/test_getargs2.py (revision 61662) +++ Lib/test/test_getargs2.py (working copy) @@ -63,6 +63,10 @@ def __int__(self): return 99 +class InvalidLongAsString: + def __int__(self): + return 'foobar' + class Unsigned_TestCase(unittest.TestCase): def test_b(self): from _testcapi import getargs_b @@ -199,6 +203,7 @@ self.failUnlessEqual(42, getargs_n(42)) self.assertRaises(OverflowError, getargs_n, VERY_LARGE) + self.assertRaises(TypeError, getargs_n, InvalidLongAsString()) class LongLong_TestCase(unittest.TestCase): def test_L(self):