summaryrefslogtreecommitdiffstats
path: root/src/responder/nss
diff options
context:
space:
mode:
authorLukas Slebodnik <lslebodn@redhat.com>2013-11-27 10:22:59 +0100
committerJakub Hrozek <jhrozek@redhat.com>2013-12-10 12:00:01 +0100
commit65f4003b93157b32eb0dcd6955d37dd009dc960e (patch)
treee8bfd8b97ca24de5acd636cd4ae4f1b84640350b /src/responder/nss
parent2b4b6b829f197493b4901bec96fefaadbc7a2464 (diff)
downloadsssd-65f4003b93157b32eb0dcd6955d37dd009dc960e.tar.gz
sssd-65f4003b93157b32eb0dcd6955d37dd009dc960e.tar.xz
sssd-65f4003b93157b32eb0dcd6955d37dd009dc960e.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/nss')
-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 b21d217ae..3fc4b6431 100644
--- a/src/responder/nss/nsssrv_netgroup.c
+++ b/src/responder/nss/nsssrv_netgroup.c
@@ -684,7 +684,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;
}