[issue9456] Apparent memory leak in PC/bdist_wininst/install.c

STINNER Victor added the comment: The distutils bdist_wininst command has been removed in Python 3.10: see bpo-42802. -- nosy: +vstinner resolution: -> wont fix stage: patch review -> resolved status: open -> closed ___ Python tracker

[issue9456] Apparent memory leak in PC/bdist_wininst/install.c

Changes by Mark Lawrence : -- nosy: -BreamoreBoy ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe: https://mail.p

[issue9456] Apparent memory leak in PC/bdist_wininst/install.c

Berker Peksag added the comment: install_c.diff looks good to me. Attaching a fresh version of it. There is a comment in PC/bdist_wininst/install.c saying IMPORTANT NOTE: IF THIS FILE IS CHANGED, PCBUILD\BDIST_WININST.VCXPROJ MUST BE REBUILT AS WELL. I don't have Windows so I can't build

[issue9456] Apparent memory leak in PC/bdist_wininst/install.c

Mark Lawrence added the comment: There is a simple patch to free memory which at a quick glance appears okay. I've not tried to apply it as the line numbers tie up with those in the existing code. -- nosy: +BreamoreBoy versions: +Python 2.7, Python 3.4, Python 3.5 -Python 3.1 ___

[issue9456] Apparent memory leak in PC/bdist_wininst/install.c

New submission from Zachary Blair : >From inspecting the code in install.c's DeleteRegistryValue() and >DeleteRegistryKey() functions, it appears as though there can be a small >memory leak in the event that either function is passed an invalid argument. This patch corrects this issue by makin