diff options
author | Lukas Slebodnik <lslebodn@redhat.com> | 2014-10-10 19:44:03 +0200 |
---|---|---|
committer | Jakub Hrozek <jhrozek@redhat.com> | 2014-10-12 17:25:47 +0200 |
commit | 609dca68b387ed6646f374545d381ca2ab8ff50a (patch) | |
tree | 56e2e2dbbdbc269ef3df5d4e3ec26430cd5bd449 | |
parent | 3db6dee5b9b24d457b4ab763e07263d121b76553 (diff) | |
download | sssd-609dca68b387ed6646f374545d381ca2ab8ff50a.tar.gz sssd-609dca68b387ed6646f374545d381ca2ab8ff50a.tar.xz sssd-609dca68b387ed6646f374545d381ca2ab8ff50a.zip |
PAC: Check return value of function hash_entries
Reviewed-by: Jakub Hrozek <jhrozek@redhat.com>
-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 9bd7e19e2..cc9259289 100644 --- a/src/responder/pac/pacsrv_cmd.c +++ b/src/responder/pac/pacsrv_cmd.c @@ -276,6 +276,14 @@ static void pac_lookup_sids_done(struct tevent_req *req) value.type = HASH_VALUE_ULONG; ret = hash_entries(pr_ctx->sid_table, &count, &entries); + if (ret != HASH_SUCCESS) { + DEBUG(SSSDBG_OP_FAILURE, "hash_entries failed [%d][%s].\n", + ret, hash_error_string(ret)); + talloc_free(pr_ctx); + pac_cmd_done(cctx, ret); + return; + } + for (c = 0; c < count; c++) { if (entries[c].value.ul == 0) { ret =responder_get_domain_by_id(cctx->rctx, |