>From c9cdc0a300b7f64e62e70e3e282bbf4e7f59fa72 Mon Sep 17 00:00:00 2001 From: Stephen Gallagher Date: Wed, 24 Sep 2014 11:00:44 -0400 Subject: [PATCH 07/24] 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) --- src/util/domain_info_utils.c | 3 --- 1 file changed, 3 deletions(-) diff --git a/src/util/domain_info_utils.c b/src/util/domain_info_utils.c index ff36d2e17fc66f5af3fdf2ce0ddbd8edad24ec6f..9ab6e4503a3ee0f8712c929e6cbfccdc6e46e6fb 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; } -- 2.4.3