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:33:56 +0200 |
| commit | 14b2def9b1bceb84ea20a49ba2f66815aac844c2 (patch) | |
| tree | 11f993c1f1b00f9a380b64d605dbe1012bfb24c3 /src/providers/ldap/ldap_id.c | |
| parent | a5c75692c17498a61babb7045db2dc3f25b0d1af (diff) | |
| download | sssd-14b2def9b1bceb84ea20a49ba2f66815aac844c2.tar.gz sssd-14b2def9b1bceb84ea20a49ba2f66815aac844c2.tar.xz sssd-14b2def9b1bceb84ea20a49ba2f66815aac844c2.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/ldap_id.c')
| -rw-r--r-- | src/providers/ldap/ldap_id.c | 2 |
1 files changed, 2 insertions, 0 deletions
diff --git a/src/providers/ldap/ldap_id.c b/src/providers/ldap/ldap_id.c index 59dfd0a5d..9fd95ce79 100644 --- a/src/providers/ldap/ldap_id.c +++ b/src/providers/ldap/ldap_id.c @@ -105,6 +105,7 @@ struct tevent_req *users_get_send(TALLOC_CTX *memctx, use_id_mapping = sdap_idmap_domain_has_algorithmic_mapping( ctx->opts->idmap_ctx, + sdom->dom->name, sdom->dom->domain_id); switch (filter_type) { case BE_FILTER_NAME: @@ -471,6 +472,7 @@ struct tevent_req *groups_get_send(TALLOC_CTX *memctx, use_id_mapping = sdap_idmap_domain_has_algorithmic_mapping( ctx->opts->idmap_ctx, + sdom->dom->name, sdom->dom->domain_id); switch(filter_type) { |