From a9fac774429097a8cf8c945fab94768fc7e87c05 Mon Sep 17 00:00:00 2001
From: Jakub Hrozek <jhrozek@redhat.com>
Date: Sat, 16 Jul 2011 17:03:11 +0200
Subject: Do not add a NULL host parsed from LDAP URI

https://fedorahosted.org/sssd/ticket/911
---
 src/providers/ldap/ldap_common.c | 9 ++++++++-
 1 file changed, 8 insertions(+), 1 deletion(-)

(limited to 'src')

diff --git a/src/providers/ldap/ldap_common.c b/src/providers/ldap/ldap_common.c
index b28a348f..f2729f3e 100644
--- a/src/providers/ldap/ldap_common.c
+++ b/src/providers/ldap/ldap_common.c
@@ -851,16 +851,23 @@ int sdap_service_init(TALLOC_CTX *memctx, struct be_ctx *ctx,
             goto done;
         }
 
+        if (lud->lud_host == NULL) {
+            DEBUG(2, ("The LDAP URI (%s) did not contain a host name\n",
+                      list[i]));
+            ldap_free_urldesc(lud);
+            continue;
+        }
+
         DEBUG(6, ("Added URI %s\n", list[i]));
 
         talloc_steal(service, list[i]);
 
         ret = be_fo_add_server(ctx, service->name,
                                lud->lud_host, lud->lud_port, list[i]);
+        ldap_free_urldesc(lud);
         if (ret) {
             goto done;
         }
-        ldap_free_urldesc(lud);
     }
 
     ret = be_fo_service_add_callback(memctx, ctx, service->name,
-- 
cgit