This issue tracker has been migrated to GitHub, and is currently read-only.
For more information, see the GitHub FAQs in the Python's Developer Guide.

Author martin.panter
Recipients docs@python, martin.panter
Date 2015-01-09.01:26:18
SpamBayes Score -1.0
Marked as misclassified Yes
Message-id <>
This simple patch documents that max_length has to be non-zero. The implementation actually uses zero as a special value to indicate max_length was not specified.

Also, I wonder what the point of the Decompressor.flush() method is. Reading the module code and zlib manual, it looks like it would return the same data as decompressor.decompress(decompressor.unconsumed_tail), except that it uses the Z_FINISH flag instead of Z_SYNC_FLUSH, so that zlib can optimize by assuming no more data is to be processed. Since unconsumed_tail is read-only and only relevant when using max_length, and flush does not support max_length, I wonder if the flush() method should be deprecated.

To further test this theory, I modified the file so that all appropriate flush() calls are equivalent to this:


and the tests all still pass.

BTW, it looks to me like zlib_Decompress_flush_impl() is setting avail_out too high (blessing unallocated memory) when the total length is more than half of UNIT_MAX, but I am not in a position to test this.
Date User Action Args
2015-01-09 01:26:21martin.pantersetrecipients: + martin.panter, docs@python
2015-01-09 01:26:21martin.pantersetmessageid: <>
2015-01-09 01:26:21martin.panterlinkissue23200 messages
2015-01-09 01:26:19martin.pantercreate