From 5455da4f944145239295a2d8344f1a7602b4454d Mon Sep 17 00:00:00 2001 From: Lukas Slebodnik Date: Sat, 24 Oct 2015 14:19:11 +0200 Subject: UTIL: Fix memory leak in switch_creds If we are already requested used then we needn't to call setreeuid(), setresgid(). But we forgot to relase local struct sss_creds *ssc, which is used for returnig saved credentials. Reviewed-by: Petr Cech --- src/util/become_user.c | 1 + 1 file changed, 1 insertion(+) diff --git a/src/util/become_user.c b/src/util/become_user.c index 7dd2c752b..c5e1439b6 100644 --- a/src/util/become_user.c +++ b/src/util/become_user.c @@ -150,6 +150,7 @@ errno_t switch_creds(TALLOC_CTX *mem_ctx, if (myuid == uid && mygid == gid) { DEBUG(SSSDBG_FUNC_DATA, "Already user [%"SPRIuid"].\n", uid); + talloc_zfree(ssc); return EOK; } -- cgit