Skip to content
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

Use CFStringCreateWithCStringNoCopy when the CFString doesn't outlast the char * #1081

Closed
wants to merge 1 commit into from

Conversation

AZero13
Copy link
Contributor

@AZero13 AZero13 commented Oct 17, 2024

The object is freed before anything happens to the pointers.

@AZero13 AZero13 force-pushed the cf branch 2 times, most recently from 7b9fc1c to 3db0ca6 Compare October 17, 2024 19:35
… the char *

The object is freed before anything happens to the pointers.
@michaelrsweet
Copy link
Member

OK, as this code works as-is, I'm not inclined to make changes and none of this code gets any real testing thanks to sandboxing restrictions on macOS.

@michaelrsweet michaelrsweet self-assigned this Oct 17, 2024
@michaelrsweet michaelrsweet added wontfix This will not be worked on platform issue Issue is specific to an OS or desktop labels Oct 17, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
platform issue Issue is specific to an OS or desktop wontfix This will not be worked on
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants