diff options
| author | Stephen Gallagher <sgallagh@redhat.com> | 2010-10-01 12:34:54 -0400 |
|---|---|---|
| committer | Stephen Gallagher <sgallagh@redhat.com> | 2010-10-04 09:10:06 -0400 |
| commit | ab45f99d35fdcf952ba56014a433bc814c56df8c (patch) | |
| tree | 8d6d7686334d45840386abf74b3b96a3e7700f09 /src/providers/ldap/ldap_auth.c | |
| parent | 5d395f5cecf4be9d9ebebb51515ad4e590dcc630 (diff) | |
| download | sssd-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/ldap_auth.c')
| -rw-r--r-- | src/providers/ldap/ldap_auth.c | 3 |
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; } |