diff options
| author | Stephen Gallagher <sgallagh@redhat.com> | 2014-09-24 11:00:44 -0400 |
|---|---|---|
| committer | Jakub Hrozek <jhrozek@redhat.com> | 2015-06-19 14:09:45 +0200 |
| commit | c6500f433ee8ee7e2543a84a480cf4ad7f18532a (patch) | |
| tree | 7b1bf2d33e36c919dc456d0066ad8deac3e40bc1 | |
| parent | ddd44781ba7526fd20e853572949718859c39fd9 (diff) | |
| download | sssd-c6500f433ee8ee7e2543a84a480cf4ad7f18532a.tar.gz sssd-c6500f433ee8ee7e2543a84a480cf4ad7f18532a.tar.xz sssd-c6500f433ee8ee7e2543a84a480cf4ad7f18532a.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.
(cherry picked from commit a2147c6c13c36c0ec056581b08a7ed7352ff9861)
| -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 ff36d2e17..9ab6e4503 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; } |