diff options
author | Ondrej Kos <okos@redhat.com> | 2013-08-28 14:41:38 +0200 |
---|---|---|
committer | Ondrej Kos <okos@redhat.com> | 2013-08-28 14:41:38 +0200 |
commit | aee6534c4bbe8a33de632a37d5fe239d075f3a9f (patch) | |
tree | bd15f264ef7c2e0d01ed2759c06d15c0cfc5abbe | |
parent | 429d685e8dd5f8a21c3dedb7a7c9d60f263744b5 (diff) | |
download | sssd-aee6534c4bbe8a33de632a37d5fe239d075f3a9f.tar.gz sssd-aee6534c4bbe8a33de632a37d5fe239d075f3a9f.tar.xz sssd-aee6534c4bbe8a33de632a37d5fe239d075f3a9f.zip |
p3
Explanation
Resolves:
https://fedorahosted.org/sssd/ticket/XXXX
-rw-r--r-- | src/providers/ldap/sdap_async_initgroups_ad.c | 32 |
1 files changed, 11 insertions, 21 deletions
diff --git a/src/providers/ldap/sdap_async_initgroups_ad.c b/src/providers/ldap/sdap_async_initgroups_ad.c index bd201f9b..57514a5b 100644 --- a/src/providers/ldap/sdap_async_initgroups_ad.c +++ b/src/providers/ldap/sdap_async_initgroups_ad.c @@ -470,29 +470,19 @@ sdap_get_ad_tokengroups_initgroups_lookup_done(struct tevent_req *subreq) continue; } - if (use_id_mapping) { - if (ret != EOK) { - DEBUG(SSSDBG_MINOR_FAILURE, - ("Could not convert SID to GID: [%s]. Skipping\n", - strerror(ret))); - continue; - } + if (use_id_mapping && ret != EOK) { + DEBUG(SSSDBG_MINOR_FAILURE, + ("Could not convert SID to GID: [%s]. Skipping\n", + strerror(ret))); + continue; + } - DEBUG(SSSDBG_TRACE_LIBS, - ("Processing membership GID [%lu]\n", - gid)); - /* Check whether this GID already exists in the sysdb */ - ret = sysdb_search_group_by_gid(tmp_ctx, state->sysdb, state->domain, - gid, attrs, &msg); - } else { - DEBUG(SSSDBG_TRACE_LIBS, - ("Processing membership group SID [%s]\n", - sid_str)); + DEBUG(SSSDBG_TRACE_LIBS, ("Processing membership group SID [%s]\n", + sid_str)); - ret = sysdb_search_group_by_sid_str(tmp_ctx, state->sysdb, - state->domain, sid_str, attrs, - &msg); - } + ret = sysdb_search_group_by_sid_str(tmp_ctx, state->sysdb, + state->domain, sid_str, attrs, + &msg); if (ret == EOK) { group_name = ldb_msg_find_attr_as_string(msg, SYSDB_NAME, NULL); |