Skip to content

Ensure GIL while releasing buffer - #27290

Merged
tacaswell merged 1 commit into
matplotlib:mainfrom
ksunden:macos_segfault
Nov 9, 2023
Merged

Ensure GIL while releasing buffer#27290
tacaswell merged 1 commit into
matplotlib:mainfrom
ksunden:macos_segfault

Conversation

@ksunden

@ksunden ksunden commented Nov 8, 2023

Copy link
Copy Markdown
Member

PR summary

This ensures the GIL is held while freeing the renderer buffer.

Fixes #27262

This became problematic in 3.12, but was likely possible in earlier python versions.

This method is handed to Obj-C code to call upon releasing, which we therefore do not fully control.

PR checklist

@QuLogic QuLogic added this to the v3.8.2 milestone Nov 8, 2023
@tacaswell tacaswell added the Release critical For bugs that make the library unusable (segfaults, incorrect plots, etc) and major regressions. label Nov 9, 2023
@tacaswell
tacaswell merged commit f4ca1ea into matplotlib:main Nov 9, 2023
meeseeksmachine pushed a commit to meeseeksmachine/matplotlib that referenced this pull request Nov 9, 2023
tacaswell added a commit that referenced this pull request Nov 9, 2023
…290-on-v3.8.x

Backport PR #27290 on branch v3.8.x (Ensure GIL while releasing buffer)
@QuLogic

QuLogic commented Nov 15, 2023

Copy link
Copy Markdown
Member

@meeseeksdev backport to v3.7.x

meeseeksmachine pushed a commit to meeseeksmachine/matplotlib that referenced this pull request Nov 15, 2023
@QuLogic QuLogic modified the milestones: v3.8.2, v3.7.4 Nov 15, 2023
QuLogic added a commit that referenced this pull request Nov 17, 2023
…290-on-v3.7.x

Backport PR #27290 on branch v3.7.x (Ensure GIL while releasing buffer)
wavebyrd pushed a commit to wavebyrd/matplotlib that referenced this pull request Mar 13, 2026
wavebyrd pushed a commit to wavebyrd/matplotlib that referenced this pull request Mar 13, 2026
…t-of-pr-27290-on-v3.7.x

Backport PR matplotlib#27290 on branch v3.7.x (Ensure GIL while releasing buffer)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

GUI: MacOSX Release critical For bugs that make the library unusable (segfaults, incorrect plots, etc) and major regressions.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Bug]: Segmentation fault when resizing on Python 3.12 and MacOS 14

3 participants