summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorLukas Slebodnik <lslebodn@redhat.com>2013-11-27 10:22:59 +0100
committerJakub Hrozek <jhrozek@redhat.com>2013-12-19 09:56:29 +0100
commit6e8c466a16db3a29063eb3dfe2141acc5f6b876b (patch)
treeaef5c6c6661918b6d97a678088a8f0ab7ad756bb
parentc72d14d5bfcafa25e578e6ecf7fecb1c6b131115 (diff)
downloadsssd-1.9.2-129.4.el6.tar.gz
sssd-1.9.2-129.4.el6.tar.xz
sssd-1.9.2-129.4.el6.zip
NSS: Fix memory leak in sss_setnetgrentsssd-1.9.2-129.4.el6
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
-rw-r--r--src/responder/nss/nsssrv_netgroup.c2
1 files changed, 1 insertions, 1 deletions
diff --git a/src/responder/nss/nsssrv_netgroup.c b/src/responder/nss/nsssrv_netgroup.c
index a0841d367..9f0fa8b6d 100644
--- a/src/responder/nss/nsssrv_netgroup.c
+++ b/src/responder/nss/nsssrv_netgroup.c
@@ -689,7 +689,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;
}