Memory leak in Reg2Py()

[luisgf]

BPO	20908
Nosy	@jcea
Files	winreg_leak_v33.patch

^{Note: these values reflect the state of the issue at the time it was migrated and might not reflect the current state.}

Show more details

GitHub fields:

assignee = 'https://github.com/jcea'
closed_at = <Date 2014-03-13.16:38:39.560>
created_at = <Date 2014-03-13.11:27:46.581>
labels = ['extension-modules', 'type-bug', 'OS-windows']
title = 'Memory leak in Reg2Py()'
updated_at = <Date 2014-03-13.18:09:10.731>
user = 'https://bugs.python.org/luisgf'

bugs.python.org fields:

activity = <Date 2014-03-13.18:09:10.731>
actor = 'python-dev'
assignee = 'jcea'
closed = True
closed_date = <Date 2014-03-13.16:38:39.560>
closer = 'python-dev'
components = ['Extension Modules', 'Windows']
creation = <Date 2014-03-13.11:27:46.581>
creator = 'luisgf'
dependencies = []
files = ['34394']
hgrepos = []
issue_num = 20908
keywords = ['patch']
message_count = 6.0
messages = ['213384', '213385', '213409', '213414', '213415', '213428']
nosy_count = 3.0
nosy_names = ['jcea', 'python-dev', 'luisgf']
pr_nums = []
priority = 'normal'
resolution = 'fixed'
stage = 'resolved'
status = 'closed'
superseder = None
type = 'behavior'
url = 'https://bugs.python.org/issue20908'
versions = ['Python 2.7', 'Python 3.3', 'Python 3.4']

[luisgf]

A memory leak can happend in Reg2Py() loosing the reference to str pointer.

See file PC/winreg.c +947

[luisgf]

Attach of patch for the 3.3.5 version.

[jcea]

I take care of this.

[jcea]

This is not actually important, since the program will finish a second after this. But cleanup is simple enough.

[python-dev]

New changeset b44a3f6676b8 by Jesus Cea in branch '2.7':
Closes bpo-20908: Memory leak in Reg2Py()
http://hg.python.org/cpython/rev/b44a3f6676b8

New changeset 07968254be96 by Jesus Cea in branch '3.3':
Closes bpo-20908: Memory leak in Reg2Py()
http://hg.python.org/cpython/rev/07968254be96

New changeset 227405961789 by Jesus Cea in branch 'default':
MERGE: Closes bpo-20908: Memory leak in Reg2Py()
http://hg.python.org/cpython/rev/227405961789

[python-dev]

New changeset 6e3e91662216 by Victor Stinner in branch 'default':
Issue bpo-20908: PyMem_Malloc() must be used with PyMem_Free(), not with free()
http://hg.python.org/cpython/rev/6e3e91662216