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_enum.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_enum.c')
-rw-r--r-- | src/providers/ldap/sdap_async_enum.c | 2 |
1 files changed, 2 insertions, 0 deletions
diff --git a/src/providers/ldap/sdap_async_enum.c b/src/providers/ldap/sdap_async_enum.c index b03c19a50..a1bc0977c 100644 --- a/src/providers/ldap/sdap_async_enum.c +++ b/src/providers/ldap/sdap_async_enum.c @@ -366,6 +366,7 @@ static struct tevent_req *enum_users_send(TALLOC_CTX *memctx, use_mapping = sdap_idmap_domain_has_algorithmic_mapping( ctx->opts->idmap_ctx, + sdom->dom->name, sdom->dom->domain_id); /* We always want to filter on objectclass and an available name */ @@ -540,6 +541,7 @@ static struct tevent_req *enum_groups_send(TALLOC_CTX *memctx, use_mapping = sdap_idmap_domain_has_algorithmic_mapping( ctx->opts->idmap_ctx, + sdom->dom->name, sdom->dom->domain_id); /* We always want to filter on objectclass and an available name */ |