FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

[3.6] bpo-34658: Fix rare subprocess prexec_fn fork error. (GH-9255) by gpshead · Pull Request #9292 · python/cpython · GitHub

/ cpython Public

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

Closed
gpshead wants to merge 2 commits into
python:3.6from
gpshead:backport-a20b6ad-3.6
Closed

[3.6] bpo-34658: Fix rare subprocess prexec_fn fork error. (GH-9255)#9292
gpshead wants to merge 2 commits into
python:3.6from
gpshead:backport-a20b6ad-3.6

Conversation

gpshead commented Sep 14, 2018
edited by bedevere-bot
Loading

Copy link
Copy Markdown
Member

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 commented Sep 14, 2018

Copy link
Copy Markdown
Member Author

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 closed this Sep 14, 2018
gpshead deleted the backport-a20b6ad-3.6 branch September 14, 2018 05:23
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
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


Back | FazBrowse Home | New Git URL