diff options
author | Sumit Bose <sbose@redhat.com> | 2013-10-24 11:45:57 +0200 |
---|---|---|
committer | Jakub Hrozek <jhrozek@redhat.com> | 2013-10-25 15:31:15 +0200 |
commit | fdda4b659fa3be3027df91a2b053835186ec2c59 (patch) | |
tree | f5f579b9d36f38df405db0f8b3552e2b4d85d21b /src/providers/ldap/sdap_async_initgroups.c | |
parent | 3cbbfb4b05d0eb0a0809704e83589d0075e117a0 (diff) | |
download | sssd-fdda4b659fa3be3027df91a2b053835186ec2c59.tar.gz sssd-fdda4b659fa3be3027df91a2b053835186ec2c59.tar.xz sssd-fdda4b659fa3be3027df91a2b053835186ec2c59.zip |
sdap_idmap_domain_has_algorithmic_mapping: add domain name argument
When libss_idmap was only used to algorithmically map a SID to a POSIX
ID a domain SID was strictly necessary and the only information needed
to find a domain.
With the introduction of external mappings there are cases where a
domain SID is not available. Currently we relied on the fact that
external mapping was always used as a default if not specific
information about the domain was found. The lead to extra CPU cycles and
potentially confusing debug messages. Adding the domain name as a search
parameter will avoid this.
Diffstat (limited to 'src/providers/ldap/sdap_async_initgroups.c')
-rw-r--r-- | src/providers/ldap/sdap_async_initgroups.c | 4 |
1 files changed, 4 insertions, 0 deletions
diff --git a/src/providers/ldap/sdap_async_initgroups.c b/src/providers/ldap/sdap_async_initgroups.c index e8de8d5c2..c16d48495 100644 --- a/src/providers/ldap/sdap_async_initgroups.c +++ b/src/providers/ldap/sdap_async_initgroups.c @@ -100,6 +100,7 @@ static errno_t sdap_add_incomplete_groups(struct sysdb_ctx *sysdb, } use_id_mapping = sdap_idmap_domain_has_algorithmic_mapping(opts->idmap_ctx, + domain->name, domain->domain_id); ret = sysdb_transaction_start(sysdb); @@ -1542,6 +1543,7 @@ static struct tevent_req *sdap_initgr_rfc2307bis_send( use_id_mapping = sdap_idmap_domain_has_algorithmic_mapping( opts->idmap_ctx, + sdom->dom->name, sdom->dom->domain_id); state->base_filter = @@ -2637,6 +2639,7 @@ struct tevent_req *sdap_get_initgr_send(TALLOC_CTX *memctx, use_id_mapping = sdap_idmap_domain_has_algorithmic_mapping( id_ctx->opts->idmap_ctx, + sdom->dom->name, sdom->dom->domain_id); ret = sss_filter_sanitize(state, name, &clean_name); @@ -2684,6 +2687,7 @@ struct tevent_req *sdap_get_initgr_send(TALLOC_CTX *memctx, state->use_id_mapping = sdap_idmap_domain_has_algorithmic_mapping( state->opts->idmap_ctx, + state->dom->name, state->dom->domain_id); ret = sdap_get_initgr_next_base(req); |