diff options
| -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); |