Skip to content

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 <[email protected]>
@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