From 2bce4118aafe7ad3033604d9f7b56e6512ad4f92 Mon Sep 17 00:00:00 2001
From: Stephen Gallagher <sgallagh@redhat.com>
Date: Wed, 24 Sep 2014 11:00:44 -0400
Subject: [PATCH 27/45] 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