From 7f987398b7fa8984ab8aa88acff2b7e1d89e3da7 Mon Sep 17 00:00:00 2001 From: Antoine Pitrou Date: Mon, 13 May 2013 19:46:29 +0200 Subject: Issue #17968: Fix memory leak in os.listxattr(). --- Misc/NEWS | 4 +++- Modules/posixmodule.c | 4 +++- 2 files changed, 6 insertions(+), 2 deletions(-) diff --git a/Misc/NEWS b/Misc/NEWS index c81aeb4..28cdf7e 100644 --- a/Misc/NEWS +++ b/Misc/NEWS @@ -49,8 +49,10 @@ Core and Builtins Library ------- +- Issue #17968: Fix memory leak in os.listxattr(). + - Issue #17606: Fixed support of encoded byte strings in the XMLGenerator - .characters() and ignorableWhitespace() methods. Original patch by Sebastian + characters() and ignorableWhitespace() methods. Original patch by Sebastian Ortiz Vasquez. - Issue #17732: Ignore distutils.cfg options pertaining to install paths if a diff --git a/Modules/posixmodule.c b/Modules/posixmodule.c index 551af2f..3e5e580 100644 --- a/Modules/posixmodule.c +++ b/Modules/posixmodule.c @@ -10625,8 +10625,10 @@ posix_listxattr(PyObject *self, PyObject *args, PyObject *kwargs) Py_END_ALLOW_THREADS; if (length < 0) { - if (errno == ERANGE) + if (errno == ERANGE) { + PyMem_FREE(buffer); continue; + } path_error("listxattr", &path); break; } -- cgit v0.12