Skip to content

gh-137335: remove a mktemp use in multiprocessing.connection to avoid security scanner noise#148578

Merged
gpshead merged 1 commit intopython:mainfrom
gpshead:no-mktemp-multiprocessing
Apr 14, 2026
Merged

gh-137335: remove a mktemp use in multiprocessing.connection to avoid security scanner noise#148578
gpshead merged 1 commit intopython:mainfrom
gpshead:no-mktemp-multiprocessing

Conversation

@gpshead
Copy link
Copy Markdown
Member

@gpshead gpshead commented Apr 14, 2026

This is not a security issue given it is a random filename in a mode 0o700 mkdtemp directory, but naive automated scanners that don't do code analysis flag it regardless. (https://github.com/python/cpython/security/code-scanning/190)

Followon to #137389 which fixed a bunch of others.

@gpshead gpshead added skip news needs backport to 3.13 bugs and security fixes needs backport to 3.14 bugs and security fixes labels Apr 14, 2026
@gpshead gpshead enabled auto-merge (squash) April 14, 2026 23:08
@gpshead gpshead merged commit fd81246 into python:main Apr 14, 2026
56 checks passed
@miss-islington-app
Copy link
Copy Markdown

Thanks @gpshead for the PR 🌮🎉.. I'm working now to backport this PR to: 3.13, 3.14.
🐍🍒⛏🤖

miss-islington pushed a commit to miss-islington/cpython that referenced this pull request Apr 14, 2026
… avoid security scanner noise (pythonGH-148578)

remove a mktemp use to avoid security scanner noise
(cherry picked from commit fd81246bd55e4fab1976a7cca3e5d42582dbdac0)

Co-authored-by: Gregory P. Smith <68491+gpshead@users.noreply.github.com>
miss-islington pushed a commit to miss-islington/cpython that referenced this pull request Apr 14, 2026
… avoid security scanner noise (pythonGH-148578)

remove a mktemp use to avoid security scanner noise
(cherry picked from commit fd81246bd55e4fab1976a7cca3e5d42582dbdac0)

Co-authored-by: Gregory P. Smith <68491+gpshead@users.noreply.github.com>
@bedevere-app
Copy link
Copy Markdown

bedevere-app bot commented Apr 14, 2026

GH-148583 is a backport of this pull request to the 3.14 branch.

@bedevere-app bedevere-app bot removed the needs backport to 3.14 bugs and security fixes label Apr 14, 2026
@bedevere-app
Copy link
Copy Markdown

bedevere-app bot commented Apr 14, 2026

GH-148584 is a backport of this pull request to the 3.13 branch.

@bedevere-app bedevere-app bot removed the needs backport to 3.13 bugs and security fixes label Apr 14, 2026
gpshead added a commit that referenced this pull request Apr 15, 2026
…o avoid security scanner noise (GH-148578) (#148583)

gh-137335: remove a mktemp use in multiprocessing.connection to avoid security scanner noise (GH-148578)

remove a mktemp use to avoid security scanner noise
(cherry picked from commit fd81246)

Co-authored-by: Gregory P. Smith <68491+gpshead@users.noreply.github.com>
gpshead added a commit that referenced this pull request Apr 15, 2026
…o avoid security scanner noise (GH-148578) (#148584)

gh-137335: remove a mktemp use in multiprocessing.connection to avoid security scanner noise (GH-148578)

remove a mktemp use to avoid security scanner noise
(cherry picked from commit fd81246)

Co-authored-by: Gregory P. Smith <68491+gpshead@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant