[security][3.4] bpo-30730: Prevent environment variables injection in subprocess on Windows. (GH-2325) by serhiy-storchaka · Pull Request #2362 · python/cpython
Navigation Menu
{{ message }}
- Notifications You must be signed in to change notification settings
- Fork 34.4k
Merged
larryhastings merged 2 commits intopython:3.4from
Jul 11, 2017Conversation
Copy link Copy Markdown
Member
serhiy-storchaka
commented
Jun 23, 2017
serhiy-storchaka
commented
Prevent passing other invalid environment variables and command arguments..
(cherry picked from commit d174d24)
the-knights-who-say-ni
added
the
CLA signed
label
Copy link Copy Markdown
mention-bot
commented
Jun 23, 2017
mention-bot commented
Jun 23, 2017@serhiy-storchaka, thanks for your PR! By analyzing the history of the files in this pull request, we identified @gvanrossum, @gpshead and @tim-one to be potential reviewers.
…s on Windows. (pythonGH-2325) Prevent passing other invalid environment variables and command arguments.. (cherry picked from commit d174d24)
serhiy-storchaka
force-pushed
the
backport-d174d24-3.4
branch
from
ed4e0c7 to
1be314a
Compare
vstinner
changed the title
[3.4] bpo-30730: Prevent environment variables injection in subprocess on Windows. (GH-2325)
[security][3.4] bpo-30730: Prevent environment variables injection in subprocess on Windows. (GH-2325)
vstinner approved these changes Jun 28, 2017
vstinner
requested review from
larryhastings and
ned-deily
zooba approved these changes Jul 11, 2017
Copy link Copy Markdown
Member
zooba
left a comment
zooba
left a comment
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
larryhastings
merged commit
fe82c46
into
python:3.4
serhiy-storchaka
deleted the
backport-d174d24-3.4
branch
vstinner
mentioned this pull request
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