Skip to content

Conversation

@nurelin
Copy link
Contributor

@nurelin nurelin commented Sep 20, 2017

Fix a memory corruption in getpath.c due to mixed memory allocators
between Py_GetPath() and Py_SetPath().

The fix use the Raw allocator to mimic the windows version.

This patch should be used from python3.6 to the current version

for more details, see the bug report and
pyinstaller/pyinstaller#2812

https://bugs.python.org/issue31532

Fix a memory corruption in getpath.c due to mixed memory allocators
between Py_GetPath() and Py_SetPath().

The fix use the Raw allocator to mimic the windows version.

This patch should be used from python3.6 to the current version

for more details, see the bug report and
  pyinstaller/pyinstaller#2812

https://bugs.python.org/issue31532
@benjaminp benjaminp merged commit 3d1e2ab into python:master Sep 21, 2017
@miss-islington
Copy link
Contributor

Thanks @nurelin for the PR, and @benjaminp for merging it 🌮🎉.. I'm working now to backport this PR to: 3.6.
🐍🍒⛏🤖

@bedevere-bot
Copy link

GH-3681 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 Sep 21, 2017
…-3679)

Fix a memory corruption in getpath.c due to mixed memory allocators
between Py_GetPath() and Py_SetPath().

The fix use the Raw allocator to mimic the windows version.

This patch should be used from python3.6 to the current version

for more details, see the bug report and
  pyinstaller/pyinstaller#2812
(cherry picked from commit 3d1e2ab)
benjaminp pushed a commit that referenced this pull request Sep 21, 2017
…H-3679) (#3681)

Fix a memory corruption in getpath.c due to mixed memory allocators
between Py_GetPath() and Py_SetPath().

The fix use the Raw allocator to mimic the windows version.

This patch should be used from python3.6 to the current version

for more details, see the bug report and
  pyinstaller/pyinstaller#2812
(cherry picked from commit 3d1e2ab)
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.

6 participants