diff options
| author | Sumit Bose <sbose@redhat.com> | 2010-12-14 10:25:20 +0100 |
|---|---|---|
| committer | Stephen Gallagher <sgallagh@redhat.com> | 2010-12-14 17:13:29 -0500 |
| commit | 6c1ec705cd9ebb49bf49778e0396e32fdf38f315 (patch) | |
| tree | 9097dd1f361662a1fca1b9fd24f5738b22dff746 /src/providers/ldap | |
| parent | 4cac4c6ebe76b3e1a40e437f9594553fdb870b81 (diff) | |
| download | sssd-6c1ec705cd9ebb49bf49778e0396e32fdf38f315.tar.gz sssd-6c1ec705cd9ebb49bf49778e0396e32fdf38f315.tar.xz sssd-6c1ec705cd9ebb49bf49778e0396e32fdf38f315.zip | |
Fix unchecked return value in sdap_parse_entry()
https://fedorahosted.org/sssd/ticket/712
Diffstat (limited to 'src/providers/ldap')
| -rw-r--r-- | src/providers/ldap/sdap.c | 6 |
1 files changed, 5 insertions, 1 deletions
diff --git a/src/providers/ldap/sdap.c b/src/providers/ldap/sdap.c index cec439e1d..573de1787 100644 --- a/src/providers/ldap/sdap.c +++ b/src/providers/ldap/sdap.c @@ -104,7 +104,11 @@ int sdap_parse_entry(TALLOC_CTX *memctx, bool store; lerrno = 0; - ldap_set_option(sh->ldap, LDAP_OPT_RESULT_CODE, &lerrno); + ret = ldap_set_option(sh->ldap, LDAP_OPT_RESULT_CODE, &lerrno); + if (ret != LDAP_OPT_SUCCESS) { + DEBUG(1, ("ldap_set_option failed [%s], ignored.\n", + ldap_err2string(ret))); + } attrs = sysdb_new_attrs(memctx); if (!attrs) return ENOMEM; |