Movatterモバイル変換


[0]ホーム

URL:


Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings

[3.12] gh-124651: Quote template strings invenv activation scripts (GH-124712)#126185

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to ourterms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Merged
vsajip merged 1 commit intopython:3.12fromvstinner:venv12
Oct 31, 2024

Conversation

vstinner
Copy link
Member

@vstinnervstinner commentedOct 30, 2024
edited by bedevere-appbot
Loading

This patch properly quotes template strings invenv activation scripts. This mitigates potential command injection.

(cherry picked from commitd48cc82)

…ythonGH-124712)This patch properly quotes template strings in `venv` activationscripts. This mitigates potential command injection.(cherry picked from commitd48cc82)
@vstinner
Copy link
MemberAuthor

Please review carefully my backport, I'm not sure of what I did :-)

@y5c4l3@vsajip: Would you mind to review my backport to 3.12?

There was a merge conflict onLib/venv/__init__.py because ofcontext.script_path = srcfile.

I also backported manually the changes to:

  • Lib/venv/scripts/common/activate
  • Lib/venv/scripts/posix/activate.csh
  • Lib/venv/scripts/posix/activate.fish

I removed quotes around__VENV_xxx__ variables.

@vsajipvsajip added the 🔨 test-with-buildbotsTest PR w/ buildbots; report in status section labelOct 30, 2024
@bedevere-bot
Copy link

🤖 New build scheduled with the buildbot fleet by@vsajip for commit1408cc9 🤖

If you want to schedule another build, you need to add the🔨 test-with-buildbots label again.

@bedevere-botbedevere-bot removed the 🔨 test-with-buildbotsTest PR w/ buildbots; report in status section labelOct 30, 2024
@vsajipvsajip merged commit8450b24 intopython:3.12Oct 31, 2024
94 of 102 checks passed
@vstinnervstinner deleted the venv12 branchOctober 31, 2024 21:47
@bedevere-app
Copy link

GH-126269 is a backport of this pull request to the3.11 branch.

@vstinner
Copy link
MemberAuthor

Thanks for the review@vsajip!

Redent0r added a commit to Redent0r/CBL-Mariner that referenced this pull requestNov 6, 2024
Taken frompython/cpython#126185 which is a 3.12 backport ofpython/cpython#124712Signed-off-by: Saul Paredes <saulparedes@microsoft.com>
Redent0r added a commit to Redent0r/CBL-Mariner that referenced this pull requestNov 6, 2024
Taken frompython/cpython#126185 which is a 3.12 backport ofpython/cpython#124712Signed-off-by: Saul Paredes <saulparedes@microsoft.com>
Redent0r added a commit to Redent0r/CBL-Mariner that referenced this pull requestNov 7, 2024
Taken frompython/cpython#126185 which is a 3.12 backport ofpython/cpython#124712Signed-off-by: Saul Paredes <saulparedes@microsoft.com>
Redent0r added a commit to Redent0r/CBL-Mariner that referenced this pull requestNov 7, 2024
Taken frompython/cpython#126185 which is a 3.12 backport ofpython/cpython#124712Signed-off-by: Saul Paredes <saulparedes@microsoft.com>
Redent0r added a commit to Redent0r/CBL-Mariner that referenced this pull requestNov 7, 2024
Taken frompython/cpython#126185 which is a 3.12 backport ofpython/cpython#124712Signed-off-by: Saul Paredes <saulparedes@microsoft.com>
Redent0r added a commit to Redent0r/CBL-Mariner that referenced this pull requestNov 26, 2024
Taken frompython/cpython#126185 which is a 3.12 backport ofpython/cpython#124712Signed-off-by: Saul Paredes <saulparedes@microsoft.com>
Redent0r added a commit to Redent0r/CBL-Mariner that referenced this pull requestDec 18, 2024
Taken frompython/cpython#126185 which is a 3.12 backport ofpython/cpython#124712Signed-off-by: Saul Paredes <saulparedes@microsoft.com>
Redent0r added a commit to Redent0r/CBL-Mariner that referenced this pull requestDec 18, 2024
Taken frompython/cpython#126185 which is a 3.12 backport ofpython/cpython#124712Signed-off-by: Saul Paredes <saulparedes@microsoft.com>
rickprice pushed a commit to ActiveState/cpython that referenced this pull requestMar 6, 2025
rickprice pushed a commit to ActiveState/cpython that referenced this pull requestMar 6, 2025
rickprice pushed a commit to ActiveState/cpython that referenced this pull requestMar 7, 2025
gentoo-bot pushed a commit to gentoo/cpython that referenced this pull requestApr 9, 2025
Sign up for freeto join this conversation on GitHub. Already have an account?Sign in to comment
Reviewers

@vsajipvsajipvsajip approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

4 participants
@vstinner@bedevere-bot@vsajip@y5c4l3

[8]ページ先頭

©2009-2025 Movatter.jp