Skip to content

Fix crash in GC: PyTuple_SET_ITEM steals reference#5

Merged
sboehmann merged 1 commit intoMeVisLab:masterfrom
iakov:patch-1
Sep 19, 2019
Merged

Fix crash in GC: PyTuple_SET_ITEM steals reference#5
sboehmann merged 1 commit intoMeVisLab:masterfrom
iakov:patch-1

Conversation

@iakov
Copy link
Copy Markdown
Contributor

@iakov iakov commented Sep 18, 2019

Py_Finalize crash is caused by references mishandling in Enums. This PR fixes the issue.

@sboehmann sboehmann merged commit 642b73e into MeVisLab:master Sep 19, 2019
@sboehmann
Copy link
Copy Markdown
Member

@iakov Thank you for the patch. Looks good and is merged now.

@f03el
Copy link
Copy Markdown

f03el commented Aug 4, 2020

Sorry to necropost, but what happened to commit 642b73e? I pulled from this repo today, and while I had that merge commit locally from earlier pulls, it seems to have vanished from this repo. GitHub says "This commit does not belong to any branch on this repository."

@iakov
Copy link
Copy Markdown
Contributor Author

iakov commented Aug 4, 2020

At least the fix is still in the upstream's master. :) 963b667

@f03el
Copy link
Copy Markdown

f03el commented Aug 4, 2020

Yeah, but if a commit exists in a branch, it's not expected to disappear since that messes up everybody's Git history.

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