Skip to content

[3.6] bpo-34658: Fix rare subprocess prexec_fn fork error. (GH-9255) #9292

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

Closed
wants to merge 2 commits into from

Conversation

gpshead
Copy link
Member

@gpshead gpshead commented Sep 14, 2018

bpo-34658: Fix a rare interpreter unhandled exception state SystemError only
seen when using subprocess with a preexec_fn while an after_parent handler has
been registered with os.register_at_fork and the fork system call fails.

https://bugs.python.org/issue34658

Cherry picked from commit a20b6ad with manual fixups for it to make sense on 3.6.

https://bugs.python.org/issue34658

[bpo-34658](https://www.bugs.python.org/issue34658): Fix a rare interpreter unhandled exception state SystemError only
seen when using subprocess with a preexec_fn while an after_parent handler has
been registered with os.register_at_fork and the fork system call fails.

https://bugs.python.org/issue34658
(cherry picked from commit a20b6ad)
manual edits to fix it up for 3.6 by gpshead.
…9255)

[bpo-34658](https://www.bugs.python.org/issue34658): Fix a rare interpreter unhandled exception state SystemError only
seen when using subprocess with a preexec_fn while an after_parent handler has
been registered with os.register_at_fork and the fork system call fails.

https://bugs.python.org/issue34658.
(cherry picked from commit a20b6ad)

Co-authored-by: Gregory P. Smith <greg@krypto.org>
@gpshead
Copy link
Member Author

gpshead commented Sep 14, 2018

looking this over, I don't believe this is actually needed on 3.6. we don't have the os.register_at_fork PyOS_BeforeFork and PyOS_AfterFork_Parent APIs in 3.6 that were the source of the potential issue.

@gpshead gpshead closed this Sep 14, 2018
@gpshead gpshead deleted the backport-a20b6ad-3.6 branch September 14, 2018 05:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
awaiting merge sprint type-bug An unexpected behavior, bug, or error
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants