Hi,

while testing Stephen's 'only one local domain' patch I found that if
the last configured domain is broken sssd terminates even if there valid
domains available. The following patch will fix this.

bye,
Sumit
>From 095e85ac304ac36aad39a0c4edb436b287770771 Mon Sep 17 00:00:00 2001
From: Sumit Bose <sb...@redhat.com>
Date: Tue, 18 Aug 2009 09:17:56 +0200
Subject: [PATCH] fix return value of confdb_get_domains

If the last configured domain is broken confdb_get_domains returns
the return value of confdb_get_domain even if there are valid domains
available.
---
 server/confdb/confdb.c |    1 +
 1 files changed, 1 insertions(+), 0 deletions(-)

diff --git a/server/confdb/confdb.c b/server/confdb/confdb.c
index 8b8dc74..a44368f 100644
--- a/server/confdb/confdb.c
+++ b/server/confdb/confdb.c
@@ -816,6 +816,7 @@ int confdb_get_domains(struct confdb_ctx *cdb,
         if (ret) {
             DEBUG(0, ("Error (%d [%s]) retrieving domain [%s], skipping!\n",
                       ret, strerror(ret), domlist[i]));
+            ret = EOK;
             continue;
         }
 
-- 
1.6.2.5

_______________________________________________
sssd-devel mailing list
sssd-devel@lists.fedorahosted.org
https://fedorahosted.org/mailman/listinfo/sssd-devel

Reply via email to