diff options
| author | Jakub Hrozek <jhrozek@redhat.com> | 2015-09-02 13:40:48 +0200 |
|---|---|---|
| committer | Jakub Hrozek <jhrozek@redhat.com> | 2015-09-21 17:04:02 +0200 |
| commit | 0561d532cf76b035b73cfed929a6896071dac407 (patch) | |
| tree | 9698735c1c7b83885f49aef1e5e78868d4c910f1 | |
| parent | 99c5f2f6ba0af6ce52be0d82ec2794bacc215742 (diff) | |
| download | sssd-0561d532cf76b035b73cfed929a6896071dac407.tar.gz sssd-0561d532cf76b035b73cfed929a6896071dac407.tar.xz sssd-0561d532cf76b035b73cfed929a6896071dac407.zip | |
SDAP: Do not set is_offline if ignore_mark_offline is set
Required for:
https://fedorahosted.org/sssd/ticket/2637
The caller of the sdap_id_op requests can set the ignore_mark_offline
flag to avoid the sdap_id_op from marking the whole back end as offline.
However, there was a small bug - the is_offline internal sdap_id_op flag
was still being set. As a consequence, the error code from the
connection was ignored and EAGAIN was always returned.
Reviewed-by: Pavel Březina <pbrezina@redhat.com>
| -rw-r--r-- | src/providers/ldap/sdap_id_op.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/src/providers/ldap/sdap_id_op.c b/src/providers/ldap/sdap_id_op.c index 508bbd2ad..0474a9cb7 100644 --- a/src/providers/ldap/sdap_id_op.c +++ b/src/providers/ldap/sdap_id_op.c @@ -567,9 +567,9 @@ static void sdap_id_op_connect_done(struct tevent_req *subreq) DEBUG(SSSDBG_CRIT_FAILURE, "Failed to connect, going offline (%d [%s])\n", ret, strerror(ret)); + is_offline = true; be_mark_offline(conn_cache->id_conn->id_ctx->be); } - is_offline = true; } if (ret == EOK) { |