Skip to content

bpo-30065: Fixed arguments validation in _posixsubprocess.fork_exec(). - #1110

Merged
serhiy-storchaka merged 2 commits into
python:masterfrom
serhiy-storchaka:_posixsubprocess-fork_exec
Apr 19, 2017
Merged

bpo-30065: Fixed arguments validation in _posixsubprocess.fork_exec().#1110
serhiy-storchaka merged 2 commits into
python:masterfrom
serhiy-storchaka:_posixsubprocess-fork_exec

Conversation

@serhiy-storchaka

Copy link
Copy Markdown
Member

No description provided.

@mention-bot

Copy link
Copy Markdown

@serhiy-storchaka, thanks for your PR! By analyzing the history of the files in this pull request, we identified @gpshead, @loewis and @tim-one to be potential reviewers.

@gpshead

gpshead commented Apr 13, 2017

Copy link
Copy Markdown
Member

LGTM - Looking only at the code I was a little surprised sequence vs tuple mattered here your explanation in https://bugs.python.org/issue30065 makes sense to me. :)

@serhiy-storchaka
serhiy-storchaka merged commit 66bffd1 into python:master Apr 19, 2017
@serhiy-storchaka
serhiy-storchaka deleted the _posixsubprocess-fork_exec branch April 19, 2017 18:12
serhiy-storchaka added a commit to serhiy-storchaka/cpython that referenced this pull request Apr 19, 2017
serhiy-storchaka added a commit that referenced this pull request Apr 19, 2017
serhiy-storchaka added a commit to serhiy-storchaka/cpython that referenced this pull request Apr 19, 2017
serhiy-storchaka added a commit that referenced this pull request Apr 19, 2017
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