diff --git a/Lib/test/test_descr.py b/Lib/test/test_descr.py index d9249b6cd2c..6484ef1a758 100644 --- a/Lib/test/test_descr.py +++ b/Lib/test/test_descr.py @@ -3966,6 +3966,13 @@ class Oops(object): o.whatever = Provoker(o) del o +def wrapper_segfault(): + # SF 927248: deeply nested wrappers could cause stack overflow + f = lambda:None + for i in xrange(1000000): + f = f.__call__ + f = None + # Fix SF #762455, segfault when sys.stdout is changed in getattr def filefault(): if verbose: @@ -4121,6 +4128,7 @@ def check(expr, x, y): def test_main(): weakref_segfault() # Must be first, somehow + wrapper_segfault() do_this_first() class_docstrings() lists() diff --git a/Misc/NEWS b/Misc/NEWS index 11add31de44..34795a438ef 100644 --- a/Misc/NEWS +++ b/Misc/NEWS @@ -12,6 +12,9 @@ What's New in Python 2.5 beta 2? Core and builtins ----------------- +- Bug #927248: Recursive method-wrapper objects can now safely + be released. + - Bug #1417699: Reject locale-specific decimal point in float() and atof(). diff --git a/Objects/descrobject.c b/Objects/descrobject.c index 606ef053049..51676f6b062 100644 --- a/Objects/descrobject.c +++ b/Objects/descrobject.c @@ -892,10 +892,12 @@ typedef struct { static void wrapper_dealloc(wrapperobject *wp) { - _PyObject_GC_UNTRACK(wp); + PyObject_GC_UnTrack(wp); + Py_TRASHCAN_SAFE_BEGIN(wp) Py_XDECREF(wp->descr); Py_XDECREF(wp->self); PyObject_GC_Del(wp); + Py_TRASHCAN_SAFE_END(wp) } static int