This issue tracker has been migrated to GitHub ,
and is currently read-only.
For more information,
see the GitHub FAQs in the Python's Developer Guide.
Created on 2010年07月07日 17:52 by stutzbach, last changed 2022年04月11日 14:57 by admin. This issue is now closed.
| Messages (3) | |||
|---|---|---|---|
| msg109486 - (view) | Author: Daniel Stutzbach (stutzbach) (Python committer) | Date: 2010年07月07日 17:52 | |
In Reg2Py() in winreg.c, in the REG_MULTI_SZ case, there's the following: wchar_t **str = (wchar_t **)malloc(sizeof(wchar_t *)*s); if (str == NULL) return PyErr_NoMemory(); However, not all of the return paths after that point call free(str). The leaking paths are pretty unusual, so I'm setting the priority to low. The leaking paths occur when PyList_New returns NULL or if the registry entry contains more than INT_MAX characters. Two other things I noticed and I'm not sure about: Is INT_MAX really the maximum number of characters for a Python unicode string? Should this be using PyMem functions instead of malloc/free? |
|||
| msg220826 - (view) | Author: Mark Lawrence (BreamoreBoy) * | Date: 2014年06月17日 13:14 | |
Can somebody answer the question about PyMem versus malloc/free functions. |
|||
| msg237517 - (view) | Author: PCManticore (Claudiu.Popa) * (Python triager) | Date: 2015年03月08日 07:28 | |
This was fixed a couple of months ago: https://hg.python.org/cpython/rev/07968254be96. Also, malloc was changed to PyMem_New a couple of weeks ago: https://hg.python.org/cpython/rev/036a2aceae93 |
|||
| History | |||
|---|---|---|---|
| Date | User | Action | Args |
| 2022年04月11日 14:57:03 | admin | set | github: 53437 |
| 2015年03月08日 07:28:46 | Claudiu.Popa | set | status: open -> closed nosy: + Claudiu.Popa messages: + msg237517 resolution: fixed stage: needs patch -> resolved |
| 2014年06月17日 18:57:36 | brian.curtin | set | nosy:
- brian.curtin |
| 2014年06月17日 13:14:28 | BreamoreBoy | set | nosy:
+ BreamoreBoy messages: + msg220826 |
| 2010年07月07日 18:20:26 | brian.curtin | set | nosy:
+ brian.curtin |
| 2010年07月07日 17:52:44 | stutzbach | create | |