diff options
author | Pavel Reichl <preichl@redhat.com> | 2014-04-09 18:00:56 +0100 |
---|---|---|
committer | Jakub Hrozek <jhrozek@redhat.com> | 2014-04-10 11:49:57 +0200 |
commit | 3c11d2418677e889f2eb38e30480e008b04c4140 (patch) | |
tree | 01a80245c6fe729683b6a1890ce90f4b886609a5 /src/responder | |
parent | a8f6a9dd171f89ecfb41822901805b5c73b5ea77 (diff) | |
download | sssd-3c11d2418677e889f2eb38e30480e008b04c4140.tar.gz sssd-3c11d2418677e889f2eb38e30480e008b04c4140.tar.xz sssd-3c11d2418677e889f2eb38e30480e008b04c4140.zip |
PAC: fix clang warning
Return value of sss_dp_get_account_recv is checked and logging is done accordingly.
Reviewed-by: Sumit Bose <sbose@redhat.com>
Diffstat (limited to 'src/responder')
-rw-r--r-- | src/responder/pac/pacsrv_cmd.c | 8 |
1 files changed, 8 insertions, 0 deletions
diff --git a/src/responder/pac/pacsrv_cmd.c b/src/responder/pac/pacsrv_cmd.c index 229c30783..9bd7e19e2 100644 --- a/src/responder/pac/pacsrv_cmd.c +++ b/src/responder/pac/pacsrv_cmd.c @@ -1084,6 +1084,14 @@ static void pac_lookup_sids_next_done(struct tevent_req *subreq) ret = sss_dp_get_account_recv(req, subreq, &err_maj, &err_min, &err_msg); + if (ret != EOK) { + DEBUG(SSSDBG_OP_FAILURE, + "Unable to get information from Data Provider\n" + "dp_error: [%u], errno: [%u], error_msg: [%s]\n", + (unsigned int)err_maj, (unsigned int)err_min, + err_msg ? err_msg : "none"); + } + talloc_zfree(subreq); talloc_zfree(err_msg); /* Errors during individual lookups are ignored. */ |