`zlib._ZlibDecompressor(zdict=...)` raises `SystemError` on a zlib-wrapped stream
维护者通常 1 天内回复
还没有人认领这个 Issue。
评估
调研方向
从 Modules/zlibmodule.c 开始：阅读第 1775–1780 行附近的 _ZlibDecompressor 构造函数和第 1535–1541 行附近的 decompress_buf，然后比较第 932–938 行附近 Decompress 如何处理 Z_NEED_DICT。为 PR #159960 的 Hypothesis 测试中记录为 ZlibDecompressorZdictTests.test_zlib_wrapper 的带封装流场景补充测试；完成标准是提供的字典能够解压该流，而缺少字典时会引发 zlib.error。
由索引模型根据 Issue 内容生成。
描述
Bug description
zlib._ZlibDecompressor accepts a zdict argument, but for a zlib-wrapped stream (positive wbits) compressed with that dictionary, decompress fails with a SystemError instead of decompressing. zlib.decompressobj handles the same input:
>>> import zlib
>>> zd = b"hello world " * 10
>>> c = zlib.compressobj(zdict=zd)
>>> s = c.compress(b"hello world hello") + c.flush()
>>> zlib.decompressobj(zdict=zd).decompress(s)
b'hello world hello'
>>> zlib._ZlibDecompressor(zdict=zd).decompress(s)
Traceback (most recent call last):
File "<python-input-5>", line 1, in <module>
zlib._ZlibDecompressor(zdict=zd).decompress(s)
SystemError: <method 'decompress' of 'zlib._ZlibDecompressor' objects> returned NULL without setting an exception
Raw deflate works, so this is specific to the wrapped format:
>>> c = zlib.compressobj(wbits=-15, zdict=zd)
>>> s = c.compress(b"hello world hello") + c.flush()
>>> zlib._ZlibDecompressor(-15, zdict=zd).decompress(s)
b'hello world hello'
Two things combine here. The constructor only applies the dictionary up front for raw deflate (wbits < 0): Modules/zlibmodule.c lines 1775-1780. For a wrapped stream zlib instead signals Z_NEED_DICT after reading the header, and decompress_buf handles that by jumping to error without setting an exception: Modules/zlibmodule.c lines 1535-1541. zlib.Decompress handles the same return by calling inflateSetDictionary and continuing: Modules/zlibmodule.c lines 932-938, which is presumably what _ZlibDecompressor should do as well (and raise zlib.error when no zdict was given, as Decompress does).
_ZlibDecompressor is private and gzip only uses it with wbits=-15, so the impact is low; but a SystemError from a C method is always a bug, and the argument is accepted for this case, so we may as well fix it.
Found by the Hypothesis tests in #159960, recorded there as ZlibDecompressorZdictTests.test_zlib_wrapper under expectedFailure.
CPython versions tested on
CPython main (4e3d5ecaad2), zlib 1.3.2
Operating systems tested on
Linux
- 主要语言
- Python
- 星标
- 77.6k
- 派生
- 38.4k
- 平均合并
- 1 天 1 小时
- 30 天内合并 PR
- 589
环境准备
在浏览器里用你自己的 GitHub 账号启动这个项目的开发容器。
- 没有 Dockerfile 或 Docker Compose 文件
- 有 Pull Request 模板
- 阅读贡献指南
从这里开始
- 先读完整个 Issue，再读项目的贡献指南。
- 在 Issue 下留言说明你要接手 —— 这能避免两个人做同样的事。
- Fork 仓库，在一个分支上完成修改。
- 提交 Pull Request，并在描述里引用这个 Issue 编号。
python/cpython 的其他 Issue
-
type-bug
难度 2/5 1-3 小时 新手友好度 72/100
维护者通常 1 天内回复
-
stdlib type-bug
难度 2/5 1-3 小时 新手友好度 78/100
维护者通常 1 天内回复
-
stdlib topic-XML type-bug
难度 2/5 1-3 小时 新手友好度 76/100
维护者通常 1 天内回复
-
Docs: problem with page "msvcrt — Useful routines from the MS VC runtime"可能重新可做 @jikekei 于 10 天前认领，目前没有进行中的 PR。 未关闭docs
难度 1/5 1 小时以内 新手友好度 85/100
python/cpython#159072 · 6 条评论 ·
维护者通常 1 天内回复
-
Document unpacking in generator expressions (PEP 798)可能已有人在做 @Marwan-verse 于 8 天前认领。 未关闭docs
难度 2/5 1-3 小时 新手友好度 62/100
python/cpython#159061 · 1 条评论 ·
维护者通常 1 天内回复
相似的 Issue
-
documentation good first issue
难度 2/5 1-3 小时 新手友好度 82/100
selimfirat/pysad#379 ·
维护者通常 1 天内回复
-
bug
难度 2/5 1-3 小时 新手友好度 75/100
open-webui/open-webui#32805 ·
维护者通常 1 天内回复
-
fix: [bug description] HTML CVE metric table shows only the last CVE for each product可能已有人在做 @Chris-wh233 今天认领。 未关闭bug
难度 2/5 1-3 小时 新手友好度 66/100
ossf/cve-bin-tool#5976 ·
维护者通常 1 天内回复
-
needs-triage
难度 2/5 1-3 小时 新手友好度 72/100
gke-labs/kube-agents#3615 ·
维护者通常 1 天内回复
-
deferred
难度 2/5 1-3 小时 新手友好度 70/100
维护者通常 1 天内回复