Skip to content

bpo-20526: Fix PyThreadState_Clear(): don't decref frame#19120

Merged
vstinner merged 1 commit intopython:masterfrom
vstinner:tstate_clear
Mar 24, 2020
Merged

bpo-20526: Fix PyThreadState_Clear(): don't decref frame#19120
vstinner merged 1 commit intopython:masterfrom
vstinner:tstate_clear

Conversation

@vstinner
Copy link
Copy Markdown
Member

@vstinner vstinner commented Mar 23, 2020

PyThreadState.frame is a borrowed reference, not a strong reference:
PyThreadState_Clear() must not call Py_CLEAR(tstate->frame).

Remove test_threading.test_warnings_at_exit(): we cannot warranty
that the Python thread state of daemon threads is cleared in a
reliable way during Python shutdown.

https://bugs.python.org/issue20526

PyThreadState.frame is a borrowed reference, not a strong reference:
PyThreadState_Clear() must not call Py_CLEAR(tstate->frame).

Remove test_threading.test_warnings_at_exit(): we cannot warranty
that the Python thread state of daemon threads is cleared in a
reliable way during Python shutdown.
@vstinner vstinner merged commit 5804f87 into python:master Mar 24, 2020
@vstinner vstinner deleted the tstate_clear branch March 24, 2020 15:32
vstinner added a commit that referenced this pull request Mar 24, 2020
…H-19136)

PyThreadState.frame is a borrowed reference, not a strong reference:
PyThreadState_Clear() must not call Py_CLEAR(tstate->frame).

Remove test_threading.test_warnings_at_exit(): we cannot warranty
that the Python thread state of daemon threads is cleared in a
reliable way during Python shutdown.

(cherry picked from commit 5804f87)
vstinner added a commit that referenced this pull request Mar 24, 2020
…H-19136) (GH-19137)

PyThreadState.frame is a borrowed reference, not a strong reference:
PyThreadState_Clear() must not call Py_CLEAR(tstate->frame).

Remove test_threading.test_warnings_at_exit(): we cannot warranty
that the Python thread state of daemon threads is cleared in a
reliable way during Python shutdown.

(cherry picked from commit 5804f87)
(cherry picked from commit e97c8b0)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants