diff options
author | Lukas Slebodnik <lslebodn@redhat.com> | 2013-11-27 10:22:59 +0100 |
---|---|---|
committer | Jakub Hrozek <jhrozek@redhat.com> | 2013-12-10 12:00:53 +0100 |
commit | 8696135e659685394937f81136521c676f0a0678 (patch) | |
tree | 9246d46266402ca1d179fc7e1d09207e1a30b0e3 /src/responder | |
parent | 3405f6cff7ba3ccab8bbc2e0d043d28ae021fcb9 (diff) | |
download | sssd-8696135e659685394937f81136521c676f0a0678.tar.gz sssd-8696135e659685394937f81136521c676f0a0678.tar.xz sssd-8696135e659685394937f81136521c676f0a0678.zip |
NSS: Fix memory leak in sss_setnetgrent
struct nss_cmd_ctx was not released in function nss_cmd_setnetgrent_done
and it wasn't used in the other function, because getnetgrent creates its own
nss_cmd_ctx context. struct nss_cmd_ctx was released after closing client
because it was allocated under client context. Memory leak is apparent with
long living clients.
Resolves:
https://fedorahosted.org/sssd/ticket/2170
Diffstat (limited to 'src/responder')
-rw-r--r-- | src/responder/nss/nsssrv_netgroup.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/src/responder/nss/nsssrv_netgroup.c b/src/responder/nss/nsssrv_netgroup.c index a1c41968d..18e857937 100644 --- a/src/responder/nss/nsssrv_netgroup.c +++ b/src/responder/nss/nsssrv_netgroup.c @@ -687,7 +687,7 @@ static void nss_cmd_setnetgrent_done(struct tevent_req *req) ((uint32_t *)body)[1] = 0; /* reserved */ } - sss_cmd_done(cmdctx->cctx, NULL); + sss_cmd_done(cmdctx->cctx, cmdctx); return; } |