diff options
| author | Rob Crittenden <rcritten@redhat.com> | 2008-06-04 13:44:08 -0400 |
|---|---|---|
| committer | Rob Crittenden <rcritten@redhat.com> | 2008-06-04 22:40:59 -0400 |
| commit | 31054b5ebd53a9bf57a1e847d1cb232eb27d3b69 (patch) | |
| tree | e6bcbba19f477fb245ee49c9945f0194c4ba66fd | |
| parent | c19b68bc5eaa989d7a8630fe90e633e31a4112a6 (diff) | |
| download | freeipa-31054b5ebd53a9bf57a1e847d1cb232eb27d3b69.tar.gz freeipa-31054b5ebd53a9bf57a1e847d1cb232eb27d3b69.tar.xz freeipa-31054b5ebd53a9bf57a1e847d1cb232eb27d3b69.zip | |
Under some conditions rl may not have been initialized so the config may error out with:
UnboundLocalError: "local variable 'rl' referenced before assignment"
This is caught and ignored but the result is that the records in DNS may
not be used at all. Initializing rl to zero fixes this.
I also convert the server list into a set to make each entry unique (and
back to a list because that is what we are supposed to return)
433506
| -rw-r--r-- | ipa-python/config.py | 3 |
1 files changed, 3 insertions, 0 deletions
diff --git a/ipa-python/config.py b/ipa-python/config.py index 4671faf9..5a9770e8 100644 --- a/ipa-python/config.py +++ b/ipa-python/config.py @@ -78,6 +78,7 @@ def __parse_config(): pass def __discover_config(): + rl = 0 try: if not config.default_realm: krbctx = krbV.default_context() @@ -163,6 +164,8 @@ def init_config(args=None): __parse_config() __discover_config() + config.default_server = list(set(config.default_server)) + if not config.default_realm: raise IPAConfigError("IPA realm not found in DNS, in the config file (/etc/ipa/ipa.conf) or on the command line.") if not config.default_server: |