diff options
author | Serhiy Storchaka <storchaka@gmail.com> | 2013-02-04 10:57:16 (GMT) |
---|---|---|
committer | Serhiy Storchaka <storchaka@gmail.com> | 2013-02-04 10:57:16 (GMT) |
commit | b6a53404b782c74bd9f7817ddb97f33642146d0c (patch) | |
tree | 79d7604ba115755b16ce1afe9c555c2f745482aa /Lib/test/test_functools.py | |
parent | a4409c18eb55e3e76110af1ac88d8a8f6001d42f (diff) | |
parent | 1d0bb9c8f97b0f4fc6717f73555576f523965e42 (diff) | |
download | cpython-b6a53404b782c74bd9f7817ddb97f33642146d0c.zip cpython-b6a53404b782c74bd9f7817ddb97f33642146d0c.tar.gz cpython-b6a53404b782c74bd9f7817ddb97f33642146d0c.tar.bz2 |
Issue #6083: Fix multiple segmentation faults occured when PyArg_ParseTuple
parses nested mutating sequence.
Diffstat (limited to 'Lib/test/test_functools.py')
-rw-r--r-- | Lib/test/test_functools.py | 22 |
1 files changed, 20 insertions, 2 deletions
diff --git a/Lib/test/test_functools.py b/Lib/test/test_functools.py index 828673c..30d7fb6 100644 --- a/Lib/test/test_functools.py +++ b/Lib/test/test_functools.py @@ -194,7 +194,25 @@ class TestPartial(object): self.assertEqual(signature(f), signature(f_copy)) class TestPartialC(BaseTestC, TestPartial): - pass + + # Issue 6083: Reference counting bug + def test_setstate_refcount(self): + class BadSequence: + def __len__(self): + return 4 + def __getitem__(self, key): + if key == 0: + return max + elif key == 1: + return tuple(range(1000000)) + elif key in (2, 3): + return {} + raise IndexError + + f = self.partial(object) + self.assertRaisesRegex(SystemError, + "new style getargs format but argument is not a tuple", + f.__setstate__, BadSequence()) class TestPartialPy(BaseTestPy, TestPartial): @@ -204,7 +222,7 @@ class TestPartialPy(BaseTestPy, TestPartial): def test_repr(self): raise unittest.SkipTest("Python implementation of partial uses own repr") -class TestPartialCSubclass(BaseTestC, TestPartial): +class TestPartialCSubclass(TestPartialC): class PartialSubclass(c_functools.partial): pass |