Skip to content

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

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

[3.8] bpo-43423 Fix IndexError in subprocess _communicate function (GH-24777)#24830
miss-islington merged 1 commit into
python:3.8from
miss-islington:backport-b4fc44b-3.8

Conversation

@miss-islington

@miss-islington miss-islington commented Mar 12, 2021

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>
@miss-islington

Copy link
Copy Markdown
Contributor Author

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

@miss-islington

Copy link
Copy Markdown
Contributor Author

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

1 similar comment
@miss-islington

Copy link
Copy Markdown
Contributor Author

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

@miss-islington

Copy link
Copy Markdown
Contributor Author

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

@miss-islington
miss-islington merged commit 1a5001c into python:3.8 Mar 12, 2021
@miss-islington
miss-islington deleted the backport-b4fc44b-3.8 branch March 12, 2021 01:52
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