Skip to content

bpo-34377: update valgrind suppressions#8729

Merged
benjaminp merged 1 commit into
python:masterfrom
PaulPrice:20180810
Aug 11, 2018
Merged

bpo-34377: update valgrind suppressions#8729
benjaminp merged 1 commit into
python:masterfrom
PaulPrice:20180810

Conversation

@PaulPrice
Copy link
Copy Markdown
Contributor

@PaulPrice PaulPrice commented Aug 10, 2018

valgrind isn't seeing PyObject_Free/PyObject_Realloc.
Using _PyObject_Free/_PyObject_Realloc works.

https://bugs.python.org/issue34377

valgrind isn't seeing PyObject_Free/PyObject_Realloc.
Using _PyObject_Free/_PyObject_Realloc works.
@miss-islington
Copy link
Copy Markdown
Contributor

Thanks @PaulPrice for the PR, and @benjaminp for merging it 🌮🎉.. I'm working now to backport this PR to: 3.6, 3.7.
🐍🍒⛏🤖 I'm not a witch! I'm not a witch!

@bedevere-bot
Copy link
Copy Markdown

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

@bedevere-bot
Copy link
Copy Markdown

GH-8734 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 Aug 11, 2018
Valgrind isn't seeing PyObject_Free/PyObject_Realloc, but using _PyObject_Free/_PyObject_Realloc works.
(cherry picked from commit db6075a)

Co-authored-by: Paul Price <price@astro.princeton.edu>
miss-islington pushed a commit to miss-islington/cpython that referenced this pull request Aug 11, 2018
Valgrind isn't seeing PyObject_Free/PyObject_Realloc, but using _PyObject_Free/_PyObject_Realloc works.
(cherry picked from commit db6075a)

Co-authored-by: Paul Price <price@astro.princeton.edu>
miss-islington added a commit that referenced this pull request Aug 11, 2018
Valgrind isn't seeing PyObject_Free/PyObject_Realloc, but using _PyObject_Free/_PyObject_Realloc works.
(cherry picked from commit db6075a)

Co-authored-by: Paul Price <price@astro.princeton.edu>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants