Skip to content

you can't call CGDataProviderCreateWithData on a stack pointer #17263

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

Conversation

lawrence-danna-apple
Copy link
Contributor

It's illegal to call CGDataProviderCreateWithData on a stack pointer.

Without this fix, matplotlib will crash on an upcoming release of Mac OS, becuase
the CGDataProvider it creates will be accessed on a different thread after
its stack allocation's lifetime has ended.

…s not allowed

Without this fix, matplotlib will crash on an upcoming release of Mac OS, becuase
the CGDataProvider it creates will be accessed on a different thread after
it's stack allocation's lifetime has ended.
@tacaswell tacaswell added this to the v3.2.2 milestone Apr 29, 2020
@tacaswell
Copy link
Member

Thanks!

@lawrence-danna-apple would you be willing to comment on #17084 ?

Copy link
Member

@QuLogic QuLogic left a comment

Choose a reason for hiding this comment

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

Someone on macOS should test this, but the (Obj)C code makes sense.

Copy link
Member

@tacaswell tacaswell left a comment

Choose a reason for hiding this comment

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

I agree, code changes look correct, but someone with an OSX machine should test this out and then merge.

Copy link
Contributor

@dopplershift dopplershift left a comment

Choose a reason for hiding this comment

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

Tested and seems to work fine for me here.

@dopplershift dopplershift merged commit d03f9e5 into matplotlib:master May 2, 2020
meeseeksmachine pushed a commit to meeseeksmachine/matplotlib that referenced this pull request May 2, 2020
QuLogic added a commit that referenced this pull request May 2, 2020
…263-on-v3.2.x

Backport PR #17263 on branch v3.2.x (you can't call CGDataProviderCreateWithData on a stack pointer)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants