diff options
author | Pavel Březina <pbrezina@redhat.com> | 2013-12-05 13:19:16 +0100 |
---|---|---|
committer | Jakub Hrozek <jhrozek@redhat.com> | 2013-12-12 13:53:47 +0100 |
commit | 0926cec5b98218131ac822e1684f9bce7aa0072c (patch) | |
tree | fa1cf541340dbafcbf245422f1355b0bd05e8d84 /src/providers/fail_over_srv.c | |
parent | 8751aace6de14f3782765a89555b65e991f340a0 (diff) | |
download | sssd-0926cec5b98218131ac822e1684f9bce7aa0072c.tar.gz sssd-0926cec5b98218131ac822e1684f9bce7aa0072c.tar.xz sssd-0926cec5b98218131ac822e1684f9bce7aa0072c.zip |
failover: check dns_domain if primary servers lookup failed
If primary servers lookup failed, dns_domain is not set.
Resolves:
https://fedorahosted.org/sssd/ticket/2173
Diffstat (limited to 'src/providers/fail_over_srv.c')
-rw-r--r-- | src/providers/fail_over_srv.c | 10 |
1 files changed, 7 insertions, 3 deletions
diff --git a/src/providers/fail_over_srv.c b/src/providers/fail_over_srv.c index 543516ef1..c27416899 100644 --- a/src/providers/fail_over_srv.c +++ b/src/providers/fail_over_srv.c @@ -302,13 +302,17 @@ static void fo_discover_servers_primary_done(struct tevent_req *subreq) } if (state->backup_domain == NULL) { + /* if there is no backup domain, we are done */ DEBUG(SSSDBG_TRACE_FUNC, ("No backup domain specified\n")); goto done; } - if (strcasecmp(state->dns_domain, state->backup_domain) == 0) { - /* primary domain was unreachable, we will use servers from backup - * domain as primary */ + if (state->dns_domain != NULL + && strcasecmp(state->dns_domain, state->backup_domain) == 0) { + /* If there was no error and dns_domain is the same as backup domain, + * it means that we were unable to resolve SRV in primary domain, but + * SRV from backup domain was resolved and those servers are considered + * to be primary. We are done. */ state->backup_servers = NULL; state->num_backup_servers = 0; |