Mailing List Archive

Change by Ma Lin <malincns@163.com>:


----------
pull_requests: +27831
pull_request: https://github.com/python/cpython/pull/29588

_______________________________________
Python tracker <report@bugs.python.org>
<https://bugs.python.org/issue41735>
_______________________________________
_______________________________________________
Python-bugs-list mailing list
Unsubscribe: https://mail.python.org/mailman/options/python-bugs-list/list-python-bugs%40lists.gossamer-threads.com

Ma Lin <malincns@163.com> added the comment:

Sorry, I found an omission.

The previous PRs fixed the bug in these methods:

zlib.Compress.compress()
zlib.Decompress.decompress()

This method also has this bug, fix in PR29587 (main/3.10) and PR29588 (3.9-):

zlib.Decompress.flush()

Attached file `test_flush.py` can reliably reproduce the bug.

This time I carefully checked bz2/lzma/zlib modules, it should be no problem anymore.

Gregory P. Smith should understand these codes, add him to nosy list.

----------
nosy: +gregory.p.smith
resolution: fixed -> later
status: closed -> open
Added file: https://bugs.python.org/file50445/test_flush.py

_______________________________________
Python tracker <report@bugs.python.org>
<https://bugs.python.org/issue41735>
_______________________________________
_______________________________________________
Python-bugs-list mailing list
Unsubscribe: https://mail.python.org/mailman/options/python-bugs-list/list-python-bugs%40lists.gossamer-threads.com

Change by Gregory P. Smith <greg@krypto.org>:


----------
assignee: -> gregory.p.smith
resolution: later ->
stage: resolved -> patch review

_______________________________________
Python tracker <report@bugs.python.org>
<https://bugs.python.org/issue41735>
_______________________________________
_______________________________________________
Python-bugs-list mailing list
Unsubscribe: https://mail.python.org/mailman/options/python-bugs-list/list-python-bugs%40lists.gossamer-threads.com

Gregory P. Smith <greg@krypto.org> added the comment:


New changeset 7edb6270a78c695e4c2ae2432797dc18105374fc by Ma Lin in branch 'main':
bpo-41735: Fix thread lock in zlib.Decompress.flush() may go wrong (GH-29587)
https://github.com/python/cpython/commit/7edb6270a78c695e4c2ae2432797dc18105374fc


----------

_______________________________________
Python tracker <report@bugs.python.org>
<https://bugs.python.org/issue41735>
_______________________________________
_______________________________________________
Python-bugs-list mailing list
Unsubscribe: https://mail.python.org/mailman/options/python-bugs-list/list-python-bugs%40lists.gossamer-threads.com

Change by miss-islington <mariatta.wijaya+miss-islington@gmail.com>:


----------
nosy: +miss-islington
nosy_count: 3.0 -> 4.0
pull_requests: +28043
pull_request: https://github.com/python/cpython/pull/29811

_______________________________________
Python tracker <report@bugs.python.org>
<https://bugs.python.org/issue41735>
_______________________________________
_______________________________________________
Python-bugs-list mailing list
Unsubscribe: https://mail.python.org/mailman/options/python-bugs-list/list-python-bugs%40lists.gossamer-threads.com

Gregory P. Smith <greg@krypto.org> added the comment:


New changeset 86c1265cdc64030c8921e0da5fcae2ac64299c26 by Ma Lin in branch '3.9':
[3.9] bpo-41735: Fix thread lock in zlib.Decompress.flush() may go wrong (GH-29588)
https://github.com/python/cpython/commit/86c1265cdc64030c8921e0da5fcae2ac64299c26


----------

_______________________________________
Python tracker <report@bugs.python.org>
<https://bugs.python.org/issue41735>
_______________________________________
_______________________________________________
Python-bugs-list mailing list
Unsubscribe: https://mail.python.org/mailman/options/python-bugs-list/list-python-bugs%40lists.gossamer-threads.com

Change by miss-islington <mariatta.wijaya+miss-islington@gmail.com>:


----------
pull_requests: +28044
pull_request: https://github.com/python/cpython/pull/29812

_______________________________________
Python tracker <report@bugs.python.org>
<https://bugs.python.org/issue41735>
_______________________________________
_______________________________________________
Python-bugs-list mailing list
Unsubscribe: https://mail.python.org/mailman/options/python-bugs-list/list-python-bugs%40lists.gossamer-threads.com

Gregory P. Smith <greg@krypto.org> added the comment:

Thanks malin!

----------
resolution: -> fixed
stage: patch review -> commit review
status: open -> closed

_______________________________________
Python tracker <report@bugs.python.org>
<https://bugs.python.org/issue41735>
_______________________________________
_______________________________________________
Python-bugs-list mailing list
Unsubscribe: https://mail.python.org/mailman/options/python-bugs-list/list-python-bugs%40lists.gossamer-threads.com

miss-islington <mariatta.wijaya+miss-islington@gmail.com> added the comment:


New changeset 57100c86baa8451a568348646834380cd425b858 by Miss Islington (bot) in branch '3.10':
[3.10] bpo-41735: Fix thread lock in zlib.Decompress.flush() may go wrong (GH-29587) (GH-29811)
https://github.com/python/cpython/commit/57100c86baa8451a568348646834380cd425b858


----------

_______________________________________
Python tracker <report@bugs.python.org>
<https://bugs.python.org/issue41735>
_______________________________________
_______________________________________________
Python-bugs-list mailing list
Unsubscribe: https://mail.python.org/mailman/options/python-bugs-list/list-python-bugs%40lists.gossamer-threads.com

Ma Lin <malincns@163.com> added the comment:

Thanks for review!

----------

_______________________________________
Python tracker <report@bugs.python.org>
<https://bugs.python.org/issue41735>
_______________________________________
_______________________________________________
Python-bugs-list mailing list
Unsubscribe: https://mail.python.org/mailman/options/python-bugs-list/list-python-bugs%40lists.gossamer-threads.com