summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorOndrej Kos <okos@redhat.com>2013-08-28 14:41:38 +0200
committerOndrej Kos <okos@redhat.com>2013-08-28 14:41:38 +0200
commitaee6534c4bbe8a33de632a37d5fe239d075f3a9f (patch)
treebd15f264ef7c2e0d01ed2759c06d15c0cfc5abbe
parent429d685e8dd5f8a21c3dedb7a7c9d60f263744b5 (diff)
downloadsssd-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.c32
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);