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

[3.9] bpo-43423 Fix IndexError in subprocess _communicate function (GH-24777) by miss-islington · Pull Request #24831 · python/cpython · GitHub

Repository navigation

[3.9] bpo-43423 Fix IndexError in subprocess _communicate function (GH-24777) - #24831

Merged
miss-islington merged 1 commit into
python:3.9from
miss-islington:backport-b4fc44b-3.9
Mar 12, 2021
Merged

miss-islington merged 1 commit into
python:3.9from
miss-islington:backport-b4fc44b-3.9

Conversation

miss-islington commented Mar 12, 2021 •
edited
Loading

Copy link
Copy Markdown
Contributor

Check to make sure stdout and stderr are not empty before selecting an item from them in Windows subprocess._communicate.

Co-authored-by: Gregory P. Smith greg@krypto.org
(cherry picked from commit b4fc44b)

Co-authored-by: Chris Griffith chris@cdgriffith.com

https://bugs.python.org/issue43423

Automerge-Triggered-By: GH:gpshead

…H-24777)

Check to make sure stdout and stderr are not empty before selecting an item from them in Windows subprocess._communicate.

Co-authored-by: Gregory P. Smith <greg@krypto.org>
(cherry picked from commit b4fc44b)

Co-authored-by: Chris Griffith <chris@cdgriffith.com>

Copy link
Copy Markdown
Contributor Author

@cdgriffith and @gpshead: Status check is done, and it's a success ✅ .

Copy link
Copy Markdown
Contributor Author

@cdgriffith and @gpshead: Status check is done, and it's a success ✅ .

Copy link
Copy Markdown
Contributor Author

@cdgriffith and @gpshead: Status check is done, and it's a success ✅ .

miss-islington merged commit ad83fde into python:3.9 Mar 12, 2021
miss-islington deleted the backport-b4fc44b-3.9 branch March 12, 2021 01:56
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

type-bug An unexpected behavior, bug, or error

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants


Back | FazBrowse Home | New Git URL