Skip to content

Reference leak in _contextvars.Context.run() #120811

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.

Sign up for GitHub

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

Jump to bottom
Closed
ZeroIntensity opened this issue Jun 20, 2024 · 0 comments
Closed

Reference leak in _contextvars.Context.run() #120811

ZeroIntensity opened this issue Jun 20, 2024 · 0 comments
Labels
type-bug An unexpected behavior, bug, or error

Comments

@ZeroIntensity
Copy link
Member

ZeroIntensity commented Jun 20, 2024 •
edited by bedevere-app bot
Loading

Bug report

Bug description:

In _contextvars.Context.run, call_result is not Py_DECREF'd if _PyContext_Exit fails. See here.

CPython versions tested on:

3.11

Operating systems tested on:

Linux

Linked PRs
@ZeroIntensity ZeroIntensity added the type-bug An unexpected behavior, bug, or error label Jun 20, 2024
@ZeroIntensity ZeroIntensity mentioned this issue Jun 20, 2024
Merged
kumaraditya303 added a commit that referenced this issue Jun 21, 2024
@ZeroIntensity @kumaraditya303
gh-120811: Fix reference leak upon _PyContext_Exit failure (#120812)
aed31be 
Co-authored-by: Kumar Aditya <[email protected]>
miss-islington pushed a commit to miss-islington/cpython that referenced this issue Jun 21, 2024
@ZeroIntensity @kumaraditya303 @miss-islington
pythongh-120811: Fix reference leak upon _PyContext_Exit failure (p…
680e76d 
…ythonGH-120812)

(cherry picked from commit aed31be)

Co-authored-by: Peter <[email protected]>
Co-authored-by: Kumar Aditya <[email protected]>
@bedevere-app bedevere-app bot mentioned this issue Jun 21, 2024
Merged
miss-islington pushed a commit to miss-islington/cpython that referenced this issue Jun 21, 2024
@ZeroIntensity @kumaraditya303 @miss-islington
pythongh-120811: Fix reference leak upon _PyContext_Exit failure (p…
1d6ab56 
…ythonGH-120812)

(cherry picked from commit aed31be)

Co-authored-by: Peter <[email protected]>
Co-authored-by: Kumar Aditya <[email protected]>
@bedevere-app bedevere-app bot mentioned this issue Jun 21, 2024
Merged
kumaraditya303 added a commit that referenced this issue Jun 22, 2024
@miss-islington @ZeroIntensity @kumaraditya303
[3.12] gh-120811: Fix reference leak upon _PyContext_Exit failure (G…
b1bccab 
…H-120812) (#120844)

gh-120811: Fix reference leak upon `_PyContext_Exit` failure (GH-120812)
(cherry picked from commit aed31be)

Co-authored-by: Peter <[email protected]>
Co-authored-by: Kumar Aditya <[email protected]>
kumaraditya303 added a commit that referenced this issue Jun 22, 2024
@miss-islington @ZeroIntensity @kumaraditya303
[3.13] gh-120811: Fix reference leak upon _PyContext_Exit failure (G…
a860b1d 
…H-120812) (#120843)

gh-120811: Fix reference leak upon `_PyContext_Exit` failure (GH-120812)
(cherry picked from commit aed31be)

Co-authored-by: Peter <[email protected]>
Co-authored-by: Kumar Aditya <[email protected]>
mrahtz pushed a commit to mrahtz/cpython that referenced this issue Jun 30, 2024
@ZeroIntensity @kumaraditya303 @mrahtz
pythongh-120811: Fix reference leak upon _PyContext_Exit failure (p…
e6c583d 
…ython#120812)

Co-authored-by: Kumar Aditya <[email protected]>
noahbkim pushed a commit to hudson-trading/cpython that referenced this issue Jul 11, 2024
@ZeroIntensity @kumaraditya303 @noahbkim
pythongh-120811: Fix reference leak upon _PyContext_Exit failure (p…
62b3b0d 
…ython#120812)

Co-authored-by: Kumar Aditya <[email protected]>
estyxx pushed a commit to estyxx/cpython that referenced this issue Jul 17, 2024
@ZeroIntensity @kumaraditya303 @estyxx
pythongh-120811: Fix reference leak upon _PyContext_Exit failure (p…
506c2f6 
…ython#120812)

Co-authored-by: Kumar Aditya <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
type-bug An unexpected behavior, bug, or error
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@ZeroIntensity @kumaraditya303