diff options
author | Pavel Březina <pbrezina@redhat.com> | 2012-03-09 13:54:36 +0100 |
---|---|---|
committer | Stephen Gallagher <sgallagh@redhat.com> | 2012-03-09 08:24:57 -0500 |
commit | 7f80db0361bfaa7b874f11eefbf9f7041cfe98e6 (patch) | |
tree | dfdda74948135bf6ae659544d78c08501b6e1e55 | |
parent | 2424e15710a3738ce0676445e5b5872d50a27735 (diff) | |
download | sssd_unused-7f80db0361bfaa7b874f11eefbf9f7041cfe98e6.tar.gz sssd_unused-7f80db0361bfaa7b874f11eefbf9f7041cfe98e6.tar.xz sssd_unused-7f80db0361bfaa7b874f11eefbf9f7041cfe98e6.zip |
Two memory leaks in sss_sudo_get_values
https://fedorahosted.org/sssd/ticket/1235
-rw-r--r-- | src/sss_client/sudo/sss_sudo.c | 5 |
1 files changed, 4 insertions, 1 deletions
diff --git a/src/sss_client/sudo/sss_sudo.c b/src/sss_client/sudo/sss_sudo.c index 32c405f3..cb3c746a 100644 --- a/src/sss_client/sudo/sss_sudo.c +++ b/src/sss_client/sudo/sss_sudo.c @@ -145,15 +145,18 @@ int sss_sudo_get_values(struct sss_sudo_rule *e, for (j = 0; j < attr->num_values; j++) { values[j] = strdup(attr->values[j]); if (values[j] == NULL) { + sss_sudo_free_values(values); return ENOMEM; } } values[attr->num_values] = NULL; + + break; } } - if (values == 0) { + if (values == NULL) { return ENOENT; } |