diff options
author | Sumit Bose <sbose@redhat.com> | 2016-04-19 15:07:18 +0200 |
---|---|---|
committer | Jakub Hrozek <jhrozek@redhat.com> | 2016-04-22 18:35:54 +0200 |
commit | 57d8b4b9254442a568838cb60ea16068965f2df0 (patch) | |
tree | 7c855aad1d1f4323aea8707d3548ee63655bc920 /src/providers | |
parent | e97d9972e8ad05dff1a3b99468ee5728c2c183a8 (diff) | |
download | sssd-57d8b4b9254442a568838cb60ea16068965f2df0.tar.gz sssd-57d8b4b9254442a568838cb60ea16068965f2df0.tar.xz sssd-57d8b4b9254442a568838cb60ea16068965f2df0.zip |
IPA: terminate properly if view name lookup fails
Since commit 5a5f1e1053415efaa99bb4d5bc7ce7ac0a95b757 the view name
lookup is the last step in the subdomain lookup request. In case of an
error the request should be finished and no previous step should be
called again.
Resolves https://fedorahosted.org/sssd/ticket/2993
Reviewed-by: Jakub Hrozek <jhrozek@redhat.com>
Diffstat (limited to 'src/providers')
-rw-r--r-- | src/providers/ipa/ipa_subdomains.c | 8 |
1 files changed, 1 insertions, 7 deletions
diff --git a/src/providers/ipa/ipa_subdomains.c b/src/providers/ipa/ipa_subdomains.c index bcc6347fa..bfe770be6 100644 --- a/src/providers/ipa/ipa_subdomains.c +++ b/src/providers/ipa/ipa_subdomains.c @@ -840,13 +840,7 @@ static void ipa_get_view_name_done(struct tevent_req *req) if (ret == EOPNOTSUPP || ret == EIO) { DEBUG(SSSDBG_TRACE_FUNC, "get_view_name request failed, looks " \ "like server does not support views.\n"); - ret = ipa_check_master(ctx); - if (ret == EAGAIN) { - return; - } else if (ret != EOK) { - goto done; - } - + ret = EOK; } else { DEBUG(SSSDBG_OP_FAILURE, "get_view_name request failed.\n"); } |