summaryrefslogtreecommitdiffstats
path: root/src/providers
diff options
context:
space:
mode:
authorSumit Bose <sbose@redhat.com>2010-12-14 10:23:07 +0100
committerStephen Gallagher <sgallagh@redhat.com>2010-12-14 17:13:29 -0500
commit4cac4c6ebe76b3e1a40e437f9594553fdb870b81 (patch)
tree1335acf69ec67471d883fcc0ba65cceb5f95d0b2 /src/providers
parentdf606157d22a633aab8cc7d532117a1b20870b58 (diff)
downloadsssd-4cac4c6ebe76b3e1a40e437f9594553fdb870b81.tar.gz
sssd-4cac4c6ebe76b3e1a40e437f9594553fdb870b81.tar.xz
sssd-4cac4c6ebe76b3e1a40e437f9594553fdb870b81.zip
Fix unchecked return value in sdap_get_msg_dn()
https://fedorahosted.org/sssd/ticket/712
Diffstat (limited to 'src/providers')
-rw-r--r--src/providers/ldap/sdap.c7
1 files changed, 6 insertions, 1 deletions
diff --git a/src/providers/ldap/sdap.c b/src/providers/ldap/sdap.c
index 1735ac520..cec439e1d 100644
--- a/src/providers/ldap/sdap.c
+++ b/src/providers/ldap/sdap.c
@@ -268,9 +268,14 @@ int sdap_get_msg_dn(TALLOC_CTX *memctx, struct sdap_handle *sh,
{
char *str;
int lerrno;
+ int ret;
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)));
+ }
str = ldap_get_dn(sh->ldap, sm->msg);
if (!str) {