From 9b911ca14d4897e517276cb9729ecc6ad6c178db Mon Sep 17 00:00:00 2001 From: Benjamin Peterson Date: Wed, 12 Jan 2011 15:49:47 +0000 Subject: [PATCH] Merged revisions 87952-87954 via svnmerge from svn+ssh://pythondev@svn.python.org/python/branches/py3k ........ r87952 | benjamin.peterson | 2011-01-12 09:24:27 -0600 (Wed, 12 Jan 2011) | 1 line move this test to test_descr; it's not abc specific ........ r87953 | benjamin.peterson | 2011-01-12 09:25:02 -0600 (Wed, 12 Jan 2011) | 1 line oops, wrong class ........ r87954 | benjamin.peterson | 2011-01-12 09:34:01 -0600 (Wed, 12 Jan 2011) | 1 line don't segfault on deleting __abstractmethods__ #10892 ........ --- Lib/test/test_abc.py | 7 ------- Lib/test/test_descr.py | 11 +++++++++++ Misc/NEWS | 3 +++ Objects/typeobject.c | 13 +++++++++++-- 4 files changed, 25 insertions(+), 9 deletions(-) diff --git a/Lib/test/test_abc.py b/Lib/test/test_abc.py index edd2c047f222..6a8c3a132742 100644 --- a/Lib/test/test_abc.py +++ b/Lib/test/test_abc.py @@ -70,13 +70,6 @@ class TestABC(unittest.TestCase): self.assertFalse(issubclass(OldstyleClass, A)) self.assertFalse(issubclass(A, OldstyleClass)) - def test_type_has_no_abstractmethods(self): - # type pretends not to have __abstractmethods__. - self.assertRaises(AttributeError, getattr, type, "__abstractmethods__") - class meta(type): - pass - self.assertRaises(AttributeError, getattr, meta, "__abstractmethods__") - def test_isinstance_class(self): class A: __metaclass__ = abc.ABCMeta diff --git a/Lib/test/test_descr.py b/Lib/test/test_descr.py index 414913dc90bd..c482cacc2bf0 100644 --- a/Lib/test/test_descr.py +++ b/Lib/test/test_descr.py @@ -4553,6 +4553,17 @@ order (MRO) for bases """ self.assertRaises(AttributeError, getattr, EvilGetattribute(), "attr") + def test_abstractmethods(self): + # type pretends not to have __abstractmethods__. + self.assertRaises(AttributeError, getattr, type, "__abstractmethods__") + class meta(type): + pass + self.assertRaises(AttributeError, getattr, meta, "__abstractmethods__") + class X(object): + pass + with self.assertRaises(AttributeError): + del X.__abstractmethods__ + class DictProxyTests(unittest.TestCase): def setUp(self): diff --git a/Misc/NEWS b/Misc/NEWS index ab47022b8019..95e31b944a2d 100644 --- a/Misc/NEWS +++ b/Misc/NEWS @@ -9,6 +9,9 @@ What's New in Python 2.7.2? Core and Builtins ----------------- +- Issue #10892: Don't segfault when trying to delete __abstractmethods__ from a + class. + - Issue #8020: Avoid a crash where the small objects allocator would read non-Python managed memory while it is being modified by another thread. Patch by Matt Bandy. diff --git a/Objects/typeobject.c b/Objects/typeobject.c index 7a11796765f9..84a755a181c8 100644 --- a/Objects/typeobject.c +++ b/Objects/typeobject.c @@ -327,8 +327,17 @@ type_set_abstractmethods(PyTypeObject *type, PyObject *value, void *context) abc.ABCMeta.__new__, so this function doesn't do anything special to update subclasses. */ - int res = PyDict_SetItemString(type->tp_dict, - "__abstractmethods__", value); + int res; + if (value != NULL) { + res = PyDict_SetItemString(type->tp_dict, "__abstractmethods__", value); + } + else { + res = PyDict_DelItemString(type->tp_dict, "__abstractmethods__"); + if (res && PyErr_ExceptionMatches(PyExc_KeyError)) { + PyErr_Format(PyExc_AttributeError, "__abstractmethods__", value); + return -1; + } + } if (res == 0) { PyType_Modified(type); if (value && PyObject_IsTrue(value)) { -- 2.47.3