summaryrefslogtreecommitdiff
path: root/src/providers/ldap
diff options
context:
space:
mode:
authorSumit Bose <sbose@redhat.com>2010-12-14 10:25:20 +0100
committerStephen Gallagher <sgallagh@redhat.com>2010-12-14 17:13:29 -0500
commit6c1ec705cd9ebb49bf49778e0396e32fdf38f315 (patch)
tree9097dd1f361662a1fca1b9fd24f5738b22dff746 /src/providers/ldap
parent4cac4c6ebe76b3e1a40e437f9594553fdb870b81 (diff)
downloadsssd-6c1ec705cd9ebb49bf49778e0396e32fdf38f315.tar.gz
sssd-6c1ec705cd9ebb49bf49778e0396e32fdf38f315.tar.bz2
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.c6
1 files changed, 5 insertions, 1 deletions
diff --git a/src/providers/ldap/sdap.c b/src/providers/ldap/sdap.c
index cec439e1..573de178 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;