diff options
| author | Andreas Schneider <asn@samba.org> | 2017-10-25 19:25:20 +0200 |
|---|---|---|
| committer | Karolin Seeger <kseeger@samba.org> | 2017-11-01 14:59:46 +0100 |
| commit | 3c45cc8a3abe2fcd7986a9db93bd83773640c864 (patch) | |
| tree | 9fcf766d0fae265419a012d3cbf6c9f48a76b13f /source4/param | |
| parent | 81047d00c909caec4211b60c8dd3a4c2e1118f49 (diff) | |
| download | samba-3c45cc8a3abe2fcd7986a9db93bd83773640c864.tar.gz | |
s4:pyparam: Fix resource leaks on error
BUG: https://bugzilla.samba.org/show_bug.cgi?id=13101
Signed-off-by: Andreas Schneider <asn@samba.org>
Reviewed-by: Jeremy Allison <jra@samba.org>
(cherry picked from commit e56626e864492831a3dbbca2d4fb8f3281547a90)
Autobuild-User(v4-7-test): Karolin Seeger <kseeger@samba.org>
Autobuild-Date(v4-7-test): Wed Nov 1 14:59:46 CET 2017 on sn-devel-144
Diffstat (limited to 'source4/param')
| -rw-r--r-- | source4/param/pyparam.c | 6 |
1 files changed, 6 insertions, 0 deletions
diff --git a/source4/param/pyparam.c b/source4/param/pyparam.c index 713e608cf1b..f16c2c0b227 100644 --- a/source4/param/pyparam.c +++ b/source4/param/pyparam.c @@ -331,6 +331,9 @@ static PyObject *py_lp_dump_a_parameter(PyObject *self, PyObject *args) if (!ret) { PyErr_Format(PyExc_RuntimeError, "Parameter %s unknown for section %s", param_name, section_name); + if (f != stdout) { + fclose(f); + } return NULL; } @@ -479,6 +482,9 @@ static PyObject *py_lp_service_dump(PyObject *self, PyObject *args) if (!PyObject_TypeCheck(py_default_service, &PyLoadparmService)) { PyErr_SetNone(PyExc_TypeError); + if (f != stdout) { + fclose(f); + } return NULL; } |