diff options
author | Stephen Gallagher <sgallagh@redhat.com> | 2014-09-24 11:00:44 -0400 |
---|---|---|
committer | Stephen Gallagher <sgallagh@redhat.com> | 2014-09-24 13:47:06 -0400 |
commit | a2147c6c13c36c0ec056581b08a7ed7352ff9861 (patch) | |
tree | 3c7c015f43a7e850832b6712af23dafeb7aa1702 | |
parent | 3e2f89726a90dfe091d7d7d3b558d97469e5d3af (diff) | |
download | sssd-a2147c6c13c36c0ec056581b08a7ed7352ff9861.tar.gz sssd-a2147c6c13c36c0ec056581b08a7ed7352ff9861.tar.xz sssd-a2147c6c13c36c0ec056581b08a7ed7352ff9861.zip |
UTIL: Do not change SSSD domains in get_domains_head
When there was more than one SSSD domain configured, actions performed
against domains later in the list would be incorrectly told to use the
first domain as the base for locating subdomains. This was because we
were rewinding the ->prev list on the sss_domain_info object, which is
only intended to be used by confdb code. The correct approach was to
use only the parent linkage, which would take us up to the top-level
domain in this SSSD domain.
-rw-r--r-- | src/util/domain_info_utils.c | 3 |
1 files changed, 0 insertions, 3 deletions
diff --git a/src/util/domain_info_utils.c b/src/util/domain_info_utils.c index 2d29743c9..8933f5235 100644 --- a/src/util/domain_info_utils.c +++ b/src/util/domain_info_utils.c @@ -34,9 +34,6 @@ struct sss_domain_info *get_domains_head(struct sss_domain_info *domain) /* get to the top level domain */ for (dom = domain; dom->parent != NULL; dom = dom->parent); - /* proceed to the list head */ - for (; dom->prev != NULL; dom = dom->prev); - return dom; } |