summaryrefslogtreecommitdiffstats
path: root/src/providers/ldap/sdap_async_groups.c
diff options
context:
space:
mode:
authorStephen Gallagher <sgallagh@redhat.com>2012-02-16 09:26:33 -0500
committerStephen Gallagher <sgallagh@redhat.com>2012-02-17 11:10:07 -0500
commit477cb0304c9fc25d40604de5d49e080ff03938e3 (patch)
treead92f704d6f78a01e9e04d6854ed6df093766875 /src/providers/ldap/sdap_async_groups.c
parentacbc134c063e92a8db1237e1444ad4ada5f54ff8 (diff)
downloadsssd-477cb0304c9fc25d40604de5d49e080ff03938e3.tar.gz
sssd-477cb0304c9fc25d40604de5d49e080ff03938e3.tar.xz
sssd-477cb0304c9fc25d40604de5d49e080ff03938e3.zip
LDAP: Ignore group member users that do not have name attributes
Instead of failing the group lookup, just skip them. This was impacting some users of ActiveDirectory where not all users had the appropriate attributes. https://fedorahosted.org/sssd/ticket/1169
Diffstat (limited to 'src/providers/ldap/sdap_async_groups.c')
-rw-r--r--src/providers/ldap/sdap_async_groups.c4
1 files changed, 2 insertions, 2 deletions
diff --git a/src/providers/ldap/sdap_async_groups.c b/src/providers/ldap/sdap_async_groups.c
index 061a99c8..aefe3538 100644
--- a/src/providers/ldap/sdap_async_groups.c
+++ b/src/providers/ldap/sdap_async_groups.c
@@ -1685,8 +1685,8 @@ static errno_t sdap_nested_group_populate_users(struct sysdb_ctx *sysdb,
opts->user_map[SDAP_AT_USER_NAME].name,
&username);
if (ret != EOK) {
- DEBUG(1, ("User entry %d has no name attribute\n", i));
- goto done;
+ DEBUG(1, ("User entry %d has no name attribute. Skipping\n", i));
+ continue;
}
ret = sysdb_attrs_get_el(users[i], SYSDB_ORIG_DN, &el);