diff options
author | Martin Kosek <mkosek@redhat.com> | 2013-01-02 10:43:49 +0100 |
---|---|---|
committer | Martin Kosek <mkosek@redhat.com> | 2013-01-02 13:28:03 +0100 |
commit | cbb12c7cc56ba8d9ee48d261fc86ddef5e323c34 (patch) | |
tree | c59ed21278e3352fb26211bfb5cd1c8b89dde41f /ipa-client/ipaclient | |
parent | 159b681c16366261edd1597524d4697ef507dc4e (diff) | |
download | freeipa-cbb12c7cc56ba8d9ee48d261fc86ddef5e323c34.tar.gz freeipa-cbb12c7cc56ba8d9ee48d261fc86ddef5e323c34.tar.xz freeipa-cbb12c7cc56ba8d9ee48d261fc86ddef5e323c34.zip |
Do not crash when Kerberos SRV record is not found
ipa-client-install crashed when IPA server realm TXT record was
configured, but the referred domain (lower-case realm value) did
not contain any Kerberos SRV record (_kerberos._udp.<realm>.)
https://fedorahosted.org/freeipa/ticket/3316
Diffstat (limited to 'ipa-client/ipaclient')
-rw-r--r-- | ipa-client/ipaclient/ipadiscovery.py | 5 |
1 files changed, 3 insertions, 2 deletions
diff --git a/ipa-client/ipaclient/ipadiscovery.py b/ipa-client/ipaclient/ipadiscovery.py index 63acf6200..2214a81ba 100644 --- a/ipa-client/ipaclient/ipadiscovery.py +++ b/ipa-client/ipaclient/ipadiscovery.py @@ -436,9 +436,10 @@ class IPADiscovery(object): kdc = self.ipadns_search_srv(domain, '_kerberos._udp', 88, break_on_first=False) - if not kdc: + if kdc: + kdc = ','.join(kdc) + else: root_logger.debug("SRV record for KDC not found! Realm: %s, SRV record: %s" % (realm, qname)) kdc = None - kdc = ','.join(kdc) return realm, kdc |