diff options
| author | Pavel Březina <pbrezina@redhat.com> | 2013-08-22 13:58:27 +0200 |
|---|---|---|
| committer | Jakub Hrozek <jhrozek@redhat.com> | 2013-08-26 11:53:49 +0200 |
| commit | 2211abf772cb8b28356e57b8c64a4328c9b2c3dc (patch) | |
| tree | 9b232ee953c9537ce4803e3e2ace14b79331fdf9 | |
| parent | 9b43a2a6462b07075d403dbd5de487cbe7ada92c (diff) | |
| download | sssd-sid2.tar.gz sssd-sid2.tar.xz sssd-sid2.zip | |
sudo: do not strdup usn on ENOENTsid2
If USN attribute is not present, we call strdup on uninitialized
variable. This may cause segfault, or if we are lucky and
usn is NULL it will return ENOMEM.
| -rw-r--r-- | src/providers/ldap/sdap_sudo_cache.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/src/providers/ldap/sdap_sudo_cache.c b/src/providers/ldap/sdap_sudo_cache.c index 5cc1dd20..de4f65c4 100644 --- a/src/providers/ldap/sdap_sudo_cache.c +++ b/src/providers/ldap/sdap_sudo_cache.c @@ -39,7 +39,7 @@ static errno_t sdap_sudo_get_usn(TALLOC_CTX *mem_ctx, } ret = sysdb_attrs_get_string(attrs, map[SDAP_AT_SUDO_USN].sys_name, &usn); - if (ret != EOK && ret != ENOENT) { + if (ret != EOK) { DEBUG(SSSDBG_MINOR_FAILURE, ("Failed to retrieve USN value: [%s]\n", strerror(ret))); |