From 6e8c466a16db3a29063eb3dfe2141acc5f6b876b Mon Sep 17 00:00:00 2001 From: Lukas Slebodnik Date: Wed, 27 Nov 2013 10:22:59 +0100 Subject: 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 --- src/responder/nss/nsssrv_netgroup.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) 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; } -- cgit