summaryrefslogtreecommitdiffstats
path: root/src/providers/ldap
diff options
context:
space:
mode:
authorStephen Gallagher <sgallagh@redhat.com>2010-10-01 12:34:54 -0400
committerStephen Gallagher <sgallagh@redhat.com>2010-10-04 09:10:06 -0400
commitab45f99d35fdcf952ba56014a433bc814c56df8c (patch)
tree8d6d7686334d45840386abf74b3b96a3e7700f09 /src/providers/ldap
parent5d395f5cecf4be9d9ebebb51515ad4e590dcc630 (diff)
downloadsssd-ab45f99d35fdcf952ba56014a433bc814c56df8c.tar.gz
sssd-ab45f99d35fdcf952ba56014a433bc814c56df8c.tar.xz
sssd-ab45f99d35fdcf952ba56014a433bc814c56df8c.zip
Return offline instead of error
When the failover code returns that there are no available servers while we are marked offline, we were returning an error to the PAM authentication code. Instead, we should return success with a result value of SDAP_UNAVAIL so that the PAM responder will mark the domain offline and attempt offline authentication.
Diffstat (limited to 'src/providers/ldap')
-rw-r--r--src/providers/ldap/ldap_auth.c3
1 files changed, 2 insertions, 1 deletions
diff --git a/src/providers/ldap/ldap_auth.c b/src/providers/ldap/ldap_auth.c
index d39803ea6..4a4bfe17f 100644
--- a/src/providers/ldap/ldap_auth.c
+++ b/src/providers/ldap/ldap_auth.c
@@ -567,7 +567,8 @@ static void auth_resolve_done(struct tevent_req *subreq)
if (ret) {
/* all servers have been tried and none
* was found good, go offline */
- tevent_req_error(req, EIO);
+ state->result = SDAP_UNAVAIL;
+ tevent_req_done(req);
return;
}