From 5e12d81215f2a7e49ce61a9513c6624cc1afa1ad Mon Sep 17 00:00:00 2001 From: Stephen Gallagher Date: Wed, 31 Mar 2010 09:10:55 -0400 Subject: Do not revert options to defaults in SSSDConfig.get_domain() There was a faulty check in get_domain() that led to the *_provider options being re-added, sometimes after options related to them had already been set. If those options had a default value, they would be overwritten by the default. Fixes: https://fedorahosted.org/sssd/ticket/441 --- src/config/SSSDConfig.py | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/config/SSSDConfig.py b/src/config/SSSDConfig.py index 2ffb397fe..d4256d1e0 100644 --- a/src/config/SSSDConfig.py +++ b/src/config/SSSDConfig.py @@ -1445,7 +1445,7 @@ class SSSDConfig(SSSDChangeConf): [domain.set_option(opt['name'], opt['value']) for opt in self.strip_comments_empty(self.options('domain/%s' % name)) - if opt not in providers] + if (opt['name'], opt['value']) not in providers] # Determine if this domain is currently active domain.active = self.is_domain_active(name) -- cgit