diff options
author | Pavel Březina <pbrezina@redhat.com> | 2012-12-14 18:58:40 +0100 |
---|---|---|
committer | Jakub Hrozek <jhrozek@redhat.com> | 2012-12-15 19:30:42 +0100 |
commit | 5f6eb9d1797b5781ad054335402b50c725961579 (patch) | |
tree | a603b2dd5c3fffd3605ff7393a1797ee1129d9e8 /src | |
parent | dd1d16b055ef8091aa77fd656b79b9c581c71ff4 (diff) | |
download | sssd-5f6eb9d1797b5781ad054335402b50c725961579.tar.gz sssd-5f6eb9d1797b5781ad054335402b50c725961579.tar.xz sssd-5f6eb9d1797b5781ad054335402b50c725961579.zip |
let ldap_chpass_uri failover work when using same hostname1.9.2-46
https://fedorahosted.org/sssd/ticket/1699
We want to continue with the next server on all errors, not only
on ETIMEDOUT.
This particullar ticket was dealing with ECONNREFUSED.
Diffstat (limited to 'src')
-rw-r--r-- | src/providers/ldap/ldap_auth.c | 15 |
1 files changed, 4 insertions, 11 deletions
diff --git a/src/providers/ldap/ldap_auth.c b/src/providers/ldap/ldap_auth.c index b78fdb8ed..b0dd30ce6 100644 --- a/src/providers/ldap/ldap_auth.c +++ b/src/providers/ldap/ldap_auth.c @@ -609,14 +609,10 @@ static void auth_connect_done(struct tevent_req *subreq) state->sdap_service->name, state->srv, PORT_NOT_WORKING); } - if (ret == ETIMEDOUT) { - if (auth_get_server(req) == NULL) { - tevent_req_error(req, ENOMEM); - } - return; - } - tevent_req_error(req, ret); + if (auth_get_server(req) == NULL) { + tevent_req_error(req, ENOMEM); + } return; } else if (state->srv) { be_fo_set_port_status(state->ctx->be, state->sdap_service->name, @@ -659,14 +655,11 @@ static void auth_bind_user_done(struct tevent_req *subreq) state->pw_expire_data = ppolicy; } talloc_zfree(subreq); - if (ret == ETIMEDOUT) { + if (ret != EOK) { if (auth_get_server(req) == NULL) { tevent_req_error(req, ENOMEM); } return; - } else if (ret != EOK) { - tevent_req_error(req, ret); - return; } tevent_req_done(req); |