Skip to content

Conversation

@sobolevn
Copy link
Member

@sobolevn sobolevn commented Nov 6, 2022

While working on some other thing, I've noticed that this call does not use proto from here:

for proto in range(pickle.HIGHEST_PROTOCOL + 1):
z = pickle.dumps(c, proto)

I think it is a typo. Fixing it!

@AlexWaygood
Copy link
Member

Should this be backported?

@kumaraditya303 kumaraditya303 added needs backport to 3.10 only security fixes needs backport to 3.11 only security fixes labels Nov 6, 2022
@kumaraditya303
Copy link
Contributor

Should this be backported?

Yes!

@AlexWaygood AlexWaygood merged commit a0bc75e into python:main Nov 6, 2022
@miss-islington
Copy link
Contributor

Thanks @sobolevn for the PR, and @AlexWaygood for merging it 🌮🎉.. I'm working now to backport this PR to: 3.10, 3.11.
🐍🍒⛏🤖

miss-islington pushed a commit to miss-islington/cpython that referenced this pull request Nov 6, 2022
(cherry picked from commit a0bc75e)

Co-authored-by: Nikita Sobolev <[email protected]>
@bedevere-bot
Copy link

GH-99157 is a backport of this pull request to the 3.11 branch.

@bedevere-bot bedevere-bot removed the needs backport to 3.11 only security fixes label Nov 6, 2022
@bedevere-bot
Copy link

GH-99158 is a backport of this pull request to the 3.10 branch.

@bedevere-bot bedevere-bot removed the needs backport to 3.10 only security fixes label Nov 6, 2022
miss-islington pushed a commit to miss-islington/cpython that referenced this pull request Nov 6, 2022
(cherry picked from commit a0bc75e)

Co-authored-by: Nikita Sobolev <[email protected]>
miss-islington added a commit that referenced this pull request Nov 6, 2022
(cherry picked from commit a0bc75e)

Co-authored-by: Nikita Sobolev <[email protected]>
miss-islington added a commit that referenced this pull request Nov 6, 2022
(cherry picked from commit a0bc75e)

Co-authored-by: Nikita Sobolev <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

skip issue skip news tests Tests in the Lib/test dir

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants