call PyErr_Clear() when ignoring error from PyNumber_Int (closes python#15516)

benjaminp · benjaminp · commit a708adfcf663 · 2013-01-02T12:21:32.000-06:00
Patch from Tom Tromey.
diff --git a/Lib/test/test_format.py b/Lib/test/test_format.py
@@ -234,6 +234,16 @@ def test_format(self):
         testformat('%g', 1.1, '1.1')
         testformat('%#g', 1.1, '1.10000')
 
+        # Regression test for http://bugs.python.org/issue15516.
+        class IntFails(object):
+            def __int__(self):
+                raise TestFailed
+            def __long__(self):
+                return 0
+
+        fst = IntFails()
+        testformat("%x", fst, '0')
+
         # Test exception for unknown format characters
         if verbose:
             print 'Testing exceptions'
diff --git a/Misc/NEWS b/Misc/NEWS
@@ -9,6 +9,9 @@ What's New in Python 2.7.4
 Core and Builtins
 -----------------
 
+- Issue #15516: Fix a bug in PyString_FromFormat where it failed to properly
+  ignore errors from a __int__() method.
+
 - Issue #16839: Fix a segfault when calling unicode() on a classic class early
   in interpreter initialization.
 
diff --git a/Objects/stringobject.c b/Objects/stringobject.c
@@ -4489,7 +4489,10 @@ PyString_Format(PyObject *format, PyObject *args)
                     }
                     else {
                         iobj = PyNumber_Int(v);
-                        if (iobj==NULL) iobj = PyNumber_Long(v);
+                        if (iobj==NULL) {
+			    PyErr_Clear();
+			    iobj = PyNumber_Long(v);
+			}
                     }
                     if (iobj!=NULL) {
                         if (PyInt_Check(iobj)) {

Original file line number	Diff line number	Diff line change
`@@ -4489,7 +4489,10 @@ PyString_Format(PyObject format, PyObject args)`
`4489`	`4489`	`}`
`4490`	`4490`	`else {`
`4491`	`4491`	`iobj = PyNumber_Int(v);`
`4492`		`- if (iobj==NULL) iobj = PyNumber_Long(v);`
	`4492`	`+ if (iobj==NULL) {`
	`4493`	`+ PyErr_Clear();`
	`4494`	`+ iobj = PyNumber_Long(v);`
	`4495`	`+ }`
`4493`	`4496`	`}`
`4494`	`4497`	`if (iobj!=NULL) {`
`4495`	`4498`	`if (PyInt_Check(iobj)) {`