Patch python3 for CVE-2025-8194 [High] - #14691
Merged
Pawel Winogrodzki (PawelWMS) merged 3 commits intoOct 3, 2025
Merged
Conversation
Kanishk Bansal (Kanishk-Bansal)
requested a review
from a team
as a code owner
September 18, 2025 03:47
| Provides: python%{majmin_nodots} = %{version}-%{release} | ||
| %if %{with_check} | ||
|
|
||
| %if 0%{?with_check} |
There was a problem hiding this comment.
rpmspec was not able to parse the spec

-
Original:
%if %{with_check}- This condition is true if the macro
with_checkis defined and non-zero. Ifwith_checkis not defined, this can produce an error like it is creating right now when we are doing rpmspec parse.
- This condition is true if the macro
-
Changed to:
%if 0%{?with_check}0%{?with_check}safely expands to0ifwith_checkis not defined (so, false), or to the value ofwith_checkif it is defined (still works as before if set).- The
?in%{?with_check}means "use the value if defined, otherwise nothing." - Prefixing with
0ensures the macro always expands to a valid number (0/1), avoiding errors if undefined.
Author
|
/azurepipelines run |
|
Azure Pipelines successfully started running 1 pipeline(s). |
Signed-off-by: Kanishk Bansal <kanbansal@microsoft.com>
Kanishk Bansal (Kanishk-Bansal)
force-pushed
the
kanbansal/python/2.0/CVE-2025-8194
branch
from
September 18, 2025 06:20
0d41e22 to
7dcb78c
Compare
Author
|
the patch applies cleanly w.r.t upstream patch python/cpython#137645 |
Author
|
|
Sudipta Pandit (realsdx)
approved these changes
Sep 18, 2025
Sudipta Pandit (realsdx)
left a comment
Contributor
There was a problem hiding this comment.
- Patch applies cleanly
- Security tag exists
- Release bump & Changelog
- Buddy Build
- No new ptest regression
Signed-off-by: Sudipta Pandit (@realsdx)
Riken Maharjan (rikenm1)
approved these changes
Sep 18, 2025
Pawel Winogrodzki (PawelWMS)
approved these changes
Oct 2, 2025
Pawel Winogrodzki (PawelWMS)
merged commit Oct 3, 2025
b174f1e
into
fasttrack/2.0
16 of 20 checks passed
Pawel Winogrodzki (PawelWMS)
deleted the
kanbansal/python/2.0/CVE-2025-8194
branch
October 3, 2025 19:35
CBL-Mariner-Bot
pushed a commit
that referenced
this pull request
Oct 3, 2025
Collaborator
|
Auto cherry-pick results: Auto cherry-pick pipeline run -> https://dev.azure.com/mariner-org/mariner/_build/results?buildId=944855&view=results |
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
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Merge Checklist
All boxes should be checked before merging the PR (just tick any boxes which don't apply to this PR)
*-staticsubpackages, etc.) have had theirReleasetag incremented../cgmanifest.json,./toolkit/scripts/toolchain/cgmanifest.json,.github/workflows/cgmanifest.json)./LICENSES-AND-NOTICES/SPECS/data/licenses.json,./LICENSES-AND-NOTICES/SPECS/LICENSES-MAP.md,./LICENSES-AND-NOTICES/SPECS/LICENSE-EXCEPTIONS.PHOTON)*.signatures.jsonfilessudo make go-tidy-allandsudo make go-test-coveragepassSummary
What does the PR accomplish, why was it needed?
Change Log
Does this affect the toolchain?
YES
Associated issues
Links to CVEs
Test Methodology