Skip to content

Conversation

@kumaraditya303
Copy link
Contributor

@kumaraditya303 kumaraditya303 commented Jul 27, 2022

Automerge-Triggered-By: GH:pablogsal

@kumaraditya303 kumaraditya303 added needs backport to 3.10 only security fixes type-crash A hard crash of the interpreter, possibly with a core dump needs backport to 3.11 only security fixes labels Jul 27, 2022
@kumaraditya303 kumaraditya303 self-assigned this Jul 27, 2022
@kumaraditya303 kumaraditya303 requested a review from pablogsal July 27, 2022 11:45
@kumaraditya303 kumaraditya303 changed the title GH-95045: gc untrack profiler before calling back into VM GH-95045: gc untrack _lsprof.Profiler before deallocating it Jul 27, 2022
Copy link
Member

@pablogsal pablogsal left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Excellent catch!

@pablogsal
Copy link
Member

I'm a bit surprised that this has not bitten us before

@miss-islington
Copy link
Contributor

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

@miss-islington miss-islington merged commit deacf39 into python:main Jul 27, 2022
@miss-islington
Copy link
Contributor

Thanks @kumaraditya303 for the PR 🌮🎉.. I'm working now to backport this PR to: 3.10, 3.11.
🐍🍒⛏🤖

@miss-islington
Copy link
Contributor

Sorry @kumaraditya303, I had trouble checking out the 3.11 backport branch.
Please backport using cherry_picker on command line.
cherry_picker deacf391d7a1b3ab49bffa16088b3500fdb4c435 3.11

@bedevere-bot
Copy link

GH-95316 is a backport of this pull request to the 3.10 branch.

@bedevere-bot bedevere-bot removed the needs backport to 3.10 only security fixes label Jul 27, 2022
miss-islington pushed a commit to miss-islington/cpython that referenced this pull request Jul 27, 2022
…ythonGH-95315)

Automerge-Triggered-By: GH:pablogsal
(cherry picked from commit deacf39)

Co-authored-by: Kumar Aditya <59607654+kumaraditya303@users.noreply.github.com>
@AlexWaygood AlexWaygood added needs backport to 3.11 only security fixes and removed needs backport to 3.11 only security fixes labels Jul 27, 2022
@miss-islington
Copy link
Contributor

Thanks @kumaraditya303 for the PR 🌮🎉.. I'm working now to backport this PR to: 3.11.
🐍🍒⛏🤖

@bedevere-bot
Copy link

GH-95317 is a backport of this pull request to the 3.11 branch.

@bedevere-bot bedevere-bot removed the needs backport to 3.11 only security fixes label Jul 27, 2022
miss-islington pushed a commit to miss-islington/cpython that referenced this pull request Jul 27, 2022
…ythonGH-95315)

Automerge-Triggered-By: GH:pablogsal
(cherry picked from commit deacf39)

Co-authored-by: Kumar Aditya <59607654+kumaraditya303@users.noreply.github.com>
@kumaraditya303 kumaraditya303 deleted the gc-lsprof branch July 27, 2022 12:11
miss-islington added a commit that referenced this pull request Jul 27, 2022
Automerge-Triggered-By: GH:pablogsal
(cherry picked from commit deacf39)

Co-authored-by: Kumar Aditya <59607654+kumaraditya303@users.noreply.github.com>
miss-islington added a commit that referenced this pull request Jul 27, 2022
Automerge-Triggered-By: GH:pablogsal
(cherry picked from commit deacf39)

Co-authored-by: Kumar Aditya <59607654+kumaraditya303@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

type-crash A hard crash of the interpreter, possibly with a core dump

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants