From 7acaaa6c6563cf3b8ab20bf6431898d20d735842 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(-) (limited to 'src/config') diff --git a/src/config/SSSDConfig.py b/src/config/SSSDConfig.py index 431dc2046..6dd8cac2b 100644 --- a/src/config/SSSDConfig.py +++ b/src/config/SSSDConfig.py @@ -1446,7 +1446,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