Skip to content

bpo-34987: Fix a possible null pointer dereference in _pickle.c's save_reduce()#9886

Merged
serhiy-storchaka merged 2 commits into
python:masterfrom
ZackerySpytz:bpo-34987-_pickle.c-save_reduce-NULL
Dec 5, 2018
Merged

bpo-34987: Fix a possible null pointer dereference in _pickle.c's save_reduce()#9886
serhiy-storchaka merged 2 commits into
python:masterfrom
ZackerySpytz:bpo-34987-_pickle.c-save_reduce-NULL

Conversation

@ZackerySpytz
Copy link
Copy Markdown
Contributor

@ZackerySpytz ZackerySpytz commented Oct 15, 2018

Comment thread Modules/_pickle.c Outdated
if (obj_class == NULL) {
return -1;
}
p = obj_class != cls; /* true iff a problem */
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Suggested change
p = obj_class != cls; /* true iff a problem */
p = obj_class != cls;

@serhiy-storchaka serhiy-storchaka merged commit 25d3897 into python:master Dec 5, 2018
@miss-islington
Copy link
Copy Markdown
Contributor

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

@bedevere-bot
Copy link
Copy Markdown

GH-10932 is a backport of this pull request to the 3.7 branch.

miss-islington pushed a commit to miss-islington/cpython that referenced this pull request Dec 5, 2018
…e_reduce(). (pythonGH-9886)

(cherry picked from commit 25d3897)

Co-authored-by: Zackery Spytz <zspytz@gmail.com>
@bedevere-bot
Copy link
Copy Markdown

GH-10933 is a backport of this pull request to the 3.6 branch.

miss-islington pushed a commit to miss-islington/cpython that referenced this pull request Dec 5, 2018
…e_reduce(). (pythonGH-9886)

(cherry picked from commit 25d3897)

Co-authored-by: Zackery Spytz <zspytz@gmail.com>
miss-islington added a commit that referenced this pull request Dec 5, 2018
…e_reduce(). (GH-9886)

(cherry picked from commit 25d3897)

Co-authored-by: Zackery Spytz <zspytz@gmail.com>
miss-islington added a commit that referenced this pull request Dec 5, 2018
…e_reduce(). (GH-9886)

(cherry picked from commit 25d3897)

Co-authored-by: Zackery Spytz <zspytz@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

skip news type-bug An unexpected behavior, bug, or error

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants