diff options
author | Nadeem Vawda <nadeem.vawda@gmail.com> | 2012-11-11 02:16:44 (GMT) |
---|---|---|
committer | Nadeem Vawda <nadeem.vawda@gmail.com> | 2012-11-11 02:16:44 (GMT) |
commit | ec6dfcffa05414e7ee29cfe88551a3d3dcdaafdc (patch) | |
tree | 084a9ec3b07d4b337ddac4535324f3e8c117264d /Lib/test/test_zlib.py | |
parent | dd1253abdd8564b095f24107547be0b8ce91e653 (diff) | |
parent | 7ee955550b27af117ddca61deb061e13423cf24b (diff) | |
download | cpython-ec6dfcffa05414e7ee29cfe88551a3d3dcdaafdc.zip cpython-ec6dfcffa05414e7ee29cfe88551a3d3dcdaafdc.tar.gz cpython-ec6dfcffa05414e7ee29cfe88551a3d3dcdaafdc.tar.bz2 |
Issue #16411: Fix a bug where zlib.decompressobj().flush() might try to access previously-freed memory.
Patch by Serhiy Storchaka.
Diffstat (limited to 'Lib/test/test_zlib.py')
-rw-r--r-- | Lib/test/test_zlib.py | 12 |
1 files changed, 12 insertions, 0 deletions
diff --git a/Lib/test/test_zlib.py b/Lib/test/test_zlib.py index f5180e0..2f6f840 100644 --- a/Lib/test/test_zlib.py +++ b/Lib/test/test_zlib.py @@ -513,6 +513,18 @@ class CompressObjectTestCase(BaseCompressTestCase, unittest.TestCase): self.assertEqual(dco.unconsumed_tail, b'') self.assertEqual(dco.unused_data, remainder) + def test_flush_with_freed_input(self): + # Issue #16411: decompressor accesses input to last decompress() call + # in flush(), even if this object has been freed in the meanwhile. + input1 = b'abcdefghijklmnopqrstuvwxyz' + input2 = b'QWERTYUIOPASDFGHJKLZXCVBNM' + data = zlib.compress(input1) + dco = zlib.decompressobj() + dco.decompress(data, 1) + del data + data = zlib.compress(input2) + self.assertEqual(dco.flush(), input1[1:]) + if hasattr(zlib.compressobj(), "copy"): def test_compresscopy(self): # Test copying a compression object |