diff options
author | Jakub Hrozek <jhrozek@redhat.com> | 2011-07-16 17:03:11 +0200 |
---|---|---|
committer | Stephen Gallagher <sgallagh@redhat.com> | 2011-08-04 09:53:18 -0400 |
commit | 6a6b58200ff6a2d929b6c2d9eabd70d26458533d (patch) | |
tree | c0559bb7c02712babbe2f0f00c50d660382360a2 /src | |
parent | 211295040f0b8b0d754ae268132b5db659fe8c42 (diff) | |
download | sssd_unused-6a6b58200ff6a2d929b6c2d9eabd70d26458533d.tar.gz sssd_unused-6a6b58200ff6a2d929b6c2d9eabd70d26458533d.tar.xz sssd_unused-6a6b58200ff6a2d929b6c2d9eabd70d26458533d.zip |
Do not add a NULL host parsed from LDAP URI
https://fedorahosted.org/sssd/ticket/911
Diffstat (limited to 'src')
-rw-r--r-- | src/providers/ldap/ldap_common.c | 9 |
1 files changed, 8 insertions, 1 deletions
diff --git a/src/providers/ldap/ldap_common.c b/src/providers/ldap/ldap_common.c index 17b85c3f..d2477bcf 100644 --- a/src/providers/ldap/ldap_common.c +++ b/src/providers/ldap/ldap_common.c @@ -841,16 +841,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, |