8000 [3.10] bpo-37788: Fix reference leak when Thread is never joined (GH-26103) by miss-islington · Pull Request #26138 · python/cpython · GitHub
[go: up one dir, main page]

Skip to content
< 8000 div class="gh-header-show ">

[3.10] bpo-37788: Fix reference leak when Thread is never joined (GH-26103) #26138

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
May 15, 2021

Conversation

miss-islington
Copy link
Contributor
@miss-islington miss-islington commented May 14, 2021

When a Thread is not joined after it has stopped, its lock may remain in the _shutdown_locks set until interpreter shutdown. If many threads are created this way, the _shutdown_locks set could therefore grow endlessly. To avoid such a situation, purge expired locks each time a new one is added or removed.
(cherry picked from commit c10c2ec)

Co-authored-by: Antoine Pitrou antoine@python.org

https://bugs.python.org/issue37788

Automerge-Triggered-By: GH:pitrou

…6103)

When a Thread is not joined after it has stopped, its lock may remain in the _shutdown_locks set until interpreter shutdown.  If many threads are created this way, the _shutdown_locks set could therefore grow endlessly.  To avoid such a situation, purge expired locks each time a new one is added or removed.
(cherry picked from commit c10c2ec)

Co-authored-by: Antoine Pitrou <antoine@python.org>
@miss-islington
Copy link
Contributor Author

@pitrou: Status check is done, and it's a success ✅ .

2 similar comments
@miss-islington
Copy link
Contributor Author

@pitrou: Status check is done, and it's a success ✅ .

@miss-islington
Copy link
Contributor Author

@pitrou: Status check is done, and it's a success ✅ .

@miss-islington
Copy link
Contributor Author

Sorry, I can't merge this PR. Reason: Pull Request is not mergeable.

@miss-islington miss-islington deleted the backport-c10c2ec-3.10 branch May 15, 2021 09:24
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.

4 participants
0