diff options
| author | Pavel Březina <pbrezina@redhat.com> | 2012-12-14 12:29:50 +0100 |
|---|---|---|
| committer | Jakub Hrozek <jhrozek@redhat.com> | 2012-12-18 17:19:36 +0100 |
| commit | b2b0f4e1e2f347c358103397112b5cd51eacfb28 (patch) | |
| tree | b92b0192b953f7d796d5b17c222935505dd10569 /src/providers/ldap | |
| parent | 6a86d676a53614c88296df222e7fa814ffb799da (diff) | |
| download | sssd-b2b0f4e1e2f347c358103397112b5cd51eacfb28.tar.gz sssd-b2b0f4e1e2f347c358103397112b5cd51eacfb28.tar.xz sssd-b2b0f4e1e2f347c358103397112b5cd51eacfb28.zip | |
check dp error in sdap_sudo_full_refresh_done()
https://fedorahosted.org/sssd/ticket/1689
Diffstat (limited to 'src/providers/ldap')
| -rw-r--r-- | src/providers/ldap/sdap_sudo.c | 11 |
1 files changed, 8 insertions, 3 deletions
diff --git a/src/providers/ldap/sdap_sudo.c b/src/providers/ldap/sdap_sudo.c index 14d4eb19c..a21935b77 100644 --- a/src/providers/ldap/sdap_sudo.c +++ b/src/providers/ldap/sdap_sudo.c @@ -615,9 +615,8 @@ static void sdap_sudo_full_refresh_done(struct tevent_req *subreq) ret = sdap_sudo_refresh_recv(state, subreq, &state->dp_error, &state->error, &highest_usn, NULL); talloc_zfree(subreq); - if (ret != EOK) { - tevent_req_error(req, ret); - return; + if (ret != EOK || state->dp_error != DP_ERR_OK || state->error != EOK) { + goto done; } state->sudo_ctx->full_refresh_done = true; @@ -639,6 +638,12 @@ static void sdap_sudo_full_refresh_done(struct tevent_req *subreq) sdap_sudo_set_usn(state->id_ctx->srv_opts, highest_usn); } +done: + if (ret != EOK) { + tevent_req_error(req, ret); + return; + } + tevent_req_done(req); } |