[Python-Dev] questions about memory management
solipsis at pitrou.net
Sat Apr 28 20:13:03 CEST 2012
On Sat, 28 Apr 2012 10:06:52 +0200 (CEST)
Julia Lawall <julia.lawall at lip6.fr> wrote:
> In Python-3.2.3/Python/import.c, in the function
> _PyImport_FixupExtensionUnicode, is any call to PyDict_DelItemString
> needed before the final failure returns?
I would say it probably does, but it would need further examination.
Some error-checking code paths in our C code base may lack proper
cleanup, especially when an error is unlikely.
Could you open an issue at http://bugs.python.org with this?
> In Python-3.2.3/Modules/ossaudiodev.c, in the function build_namelists, is
> it intentional that labels is not freed in the last failure case:
The successful call to PyModule_AddObject() steals a reference to
`labels`, so it doesn't need to be decrefed again (the reference is
not owned by the init function anymore).
> if (PyModule_AddObject(module, "control_labels", labels) == -1)
> goto error2;
> if (PyModule_AddObject(module, "control_names", names) == -1)
> goto error1;
> return 0;
> return -1;
More information about the Python-Dev