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

[3.9] gh-97897: Prevent os.mkfifo and os.mknod segfaults with macOS 13 SDK (GH-97944) #97968

Merged
merged 1 commit into from
Oct 6, 2022

Conversation

miss-islington
Copy link
Contributor

@miss-islington miss-islington commented Oct 6, 2022

The macOS 13 SDK includes support for the mkfifoat and mknodat system calls.
Using the dir_fd option with either os.mkfifo or os.mknod could result in a
segfault if cpython is built with the macOS 13 SDK but run on an earlier
version of macOS. Prevent this by adding runtime support for detection of
these system calls ("weaklinking") as is done for other newer syscalls on
macOS.
(cherry picked from commit 6d0a019)

Co-authored-by: Ned Deily [email protected]

…3 SDK (pythonGH-97944)

The macOS 13 SDK includes support for the `mkfifoat` and `mknodat` system calls.
 Using the `dir_fd` option with either `os.mkfifo` or `os.mknod` could result in a
 segfault if cpython is built with the macOS 13 SDK but run on an earlier
 version of macOS. Prevent this by adding runtime support for detection of
 these system calls ("weaklinking") as is done for other newer syscalls on
 macOS.
(cherry picked from commit 6d0a019)

Co-authored-by: Ned Deily <[email protected]>
@bedevere-bot bedevere-bot added the type-crash A hard crash of the interpreter, possibly with a core dump label Oct 6, 2022
@ambv ambv merged commit 77796d0 into python:3.9 Oct 6, 2022
@miss-islington miss-islington deleted the backport-6d0a019-3.9 branch October 6, 2022 19:14
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.

4 participants