summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorLukas Slebodnik <lslebodn@redhat.com>2013-10-30 14:55:13 +0100
committerJakub Hrozek <jhrozek@redhat.com>2013-10-30 16:56:32 +0100
commited2f9b9b444f146429a28d02b1e3b7c97a5cfad7 (patch)
treea497caf43f4f305f8a879e7c6d27ea92bfca0791
parentf1be4caee5d9681c3f5f6d4b13c8abc8295c901b (diff)
downloadsssd-ed2f9b9b444f146429a28d02b1e3b7c97a5cfad7.tar.gz
sssd-ed2f9b9b444f146429a28d02b1e3b7c97a5cfad7.tar.xz
sssd-ed2f9b9b444f146429a28d02b1e3b7c97a5cfad7.zip
LDAP: Return correct error code
If talloc_array return NULL we should return right error code from function sdap_domain_subdom_add. It might happen that we could return either wrong error code or uninitialized variable ret.
-rw-r--r--src/providers/ldap/ldap_common.c2
1 files changed, 1 insertions, 1 deletions
diff --git a/src/providers/ldap/ldap_common.c b/src/providers/ldap/ldap_common.c
index fcb8c4884..7192740b6 100644
--- a/src/providers/ldap/ldap_common.c
+++ b/src/providers/ldap/ldap_common.c
@@ -134,7 +134,7 @@ sdap_domain_subdom_add(struct sdap_id_ctx *sdap_id_ctx,
talloc_zfree(sdom->search_bases);
sdom->search_bases = talloc_array(sdom, struct sdap_search_base *, 2);
if (sdom->search_bases == NULL) {
- return ret;
+ return ENOMEM;
}
sdom->search_bases[1] = NULL;