diff options
| author | Lukas Slebodnik <lslebodn@redhat.com> | 2015-10-24 14:19:11 +0200 |
|---|---|---|
| committer | Jakub Hrozek <jhrozek@redhat.com> | 2015-11-11 12:10:34 +0100 |
| commit | 5455da4f944145239295a2d8344f1a7602b4454d (patch) | |
| tree | 15b1519478b11b814e90e944c29bb115372799aa | |
| parent | 788146c3e3a564f333f39a2fcffccf3012cc2679 (diff) | |
| download | sssd-5455da4f944145239295a2d8344f1a7602b4454d.tar.gz sssd-5455da4f944145239295a2d8344f1a7602b4454d.tar.xz sssd-5455da4f944145239295a2d8344f1a7602b4454d.zip | |
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 <pcech@redhat.com>
| -rw-r--r-- | src/util/become_user.c | 1 |
1 files changed, 1 insertions, 0 deletions
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; } |