Skip to content

gh-148418: Fix a possible reference leak in a corrupted TYPE_CODE marshal stream#148419

Merged
serhiy-storchaka merged 3 commits intopython:mainfrom
Wulian233:marshal
May 4, 2026
Merged

gh-148418: Fix a possible reference leak in a corrupted TYPE_CODE marshal stream#148419
serhiy-storchaka merged 3 commits intopython:mainfrom
Wulian233:marshal

Conversation

@Wulian233
Copy link
Copy Markdown
Contributor

@Wulian233 Wulian233 commented Apr 12, 2026 •
edited by bedevere-app Bot
Loading

@bedevere-app bedevere-app Bot mentioned this pull request Apr 12, 2026
Closed
aisk
aisk reviewed Apr 12, 2026
View reviewed changes
Comment thread Python/marshal.c
serhiy-storchaka
serhiy-storchaka reviewed Apr 13, 2026
View reviewed changes
Comment thread Misc/NEWS.d/next/Core_and_Builtins/2026-04-12-10-40-57.gh-issue-148418.ggA1LZ.rst Outdated
@Wulian233 Wulian233 requested a review from picnixz May 2, 2026 08:05
serhiy-storchaka
serhiy-storchaka approved these changes May 4, 2026
View reviewed changes
Copy link
Copy Markdown
Member

@serhiy-storchaka serhiy-storchaka left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM. Thank you for your contribution @Wulian233.

@serhiy-storchaka serhiy-storchaka merged commit c3972f2 into python:main May 4, 2026
58 checks passed
@serhiy-storchaka serhiy-storchaka added needs backport to 3.13 bugs and security fixes needs backport to 3.14 bugs and security fixes labels May 4, 2026
@miss-islington-app
Copy link
Copy Markdown

miss-islington-app Bot commented May 4, 2026

Thanks @Wulian233 for the PR, and @serhiy-storchaka for merging it 🌮🎉.. I'm working now to backport this PR to: 3.13.
🐍🍒⛏🤖

@miss-islington-app
Copy link
Copy Markdown

miss-islington-app Bot commented May 4, 2026

Thanks @Wulian233 for the PR, and @serhiy-storchaka for merging it 🌮🎉.. I'm working now to backport this PR to: 3.14.
🐍🍒⛏🤖

@bedevere-app
Copy link
Copy Markdown

bedevere-app Bot commented May 4, 2026

GH-149363 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 May 4, 2026
@bedevere-app
Copy link
Copy Markdown

bedevere-app Bot commented May 4, 2026

GH-149364 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 May 4, 2026
@Wulian233 Wulian233 deleted the marshal branch May 4, 2026 13:46
serhiy-storchaka pushed a commit that referenced this pull request May 4, 2026
@miss-islington @Wulian233
[3.13] gh-148418: Fix a possible reference leak in a corrupted TYPE_C…
6d6471b 
…ODE marshal stream (GH-148419) (GH-149364)

(cherry picked from commit c3972f2)

Co-authored-by: Wulian233 <1055917385@qq.com>
serhiy-storchaka pushed a commit that referenced this pull request May 4, 2026
@miss-islington @Wulian233
[3.14] gh-148418: Fix a possible reference leak in a corrupted TYPE_C…
95979d1 
…ODE marshal stream (GH-148419) (GH-149363)

(cherry picked from commit c3972f2)

Co-authored-by: Wulian233 <1055917385@qq.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@aisk aisk aisk approved these changes

@serhiy-storchaka serhiy-storchaka serhiy-storchaka approved these changes

@picnixz picnixz Awaiting requested review from picnixz

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

@Wulian233 @aisk @serhiy-storchaka @picnixz