summaryrefslogtreecommitdiffstats
path: root/src/responder/pac
diff options
context:
space:
mode:
authorLukas Slebodnik <lslebodn@redhat.com>2014-10-10 19:44:03 +0200
committerJakub Hrozek <jhrozek@redhat.com>2014-10-12 17:25:47 +0200
commit609dca68b387ed6646f374545d381ca2ab8ff50a (patch)
tree56e2e2dbbdbc269ef3df5d4e3ec26430cd5bd449 /src/responder/pac
parent3db6dee5b9b24d457b4ab763e07263d121b76553 (diff)
downloadsssd-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>
Diffstat (limited to 'src/responder/pac')
-rw-r--r--src/responder/pac/pacsrv_cmd.c8
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,