summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorStephen Gallagher <sgallagh@redhat.com>2010-06-09 10:40:49 -0400
committerStephen Gallagher <sgallagh@redhat.com>2010-06-10 10:17:38 -0400
commit361e86478ff1cc7ac08981b721f508363291028c (patch)
tree9da9b542332bc86d108cc889aea4ab84ded0dd7c
parent476b2f491e248b182c0f6c3461028b61a8a87ab7 (diff)
downloadsssd-361e86478ff1cc7ac08981b721f508363291028c.tar.gz
sssd-361e86478ff1cc7ac08981b721f508363291028c.tar.bz2
sssd-361e86478ff1cc7ac08981b721f508363291028c.zip
Check return code of hash_delete in proxy_child_destructor
We can't do much about an error here, but we should be reporting it. https://fedorahosted.org/sssd/ticket/534
-rw-r--r--src/providers/proxy/proxy.c8
1 files changed, 7 insertions, 1 deletions
diff --git a/src/providers/proxy/proxy.c b/src/providers/proxy/proxy.c
index f6e7d7bb..f2131425 100644
--- a/src/providers/proxy/proxy.c
+++ b/src/providers/proxy/proxy.c
@@ -190,11 +190,17 @@ static int proxy_child_destructor(TALLOC_CTX *ctx)
struct proxy_child_ctx *child_ctx =
talloc_get_type(ctx, struct proxy_child_ctx);
hash_key_t key;
+ int hret;
DEBUG(8, ("Removing proxy child id [%d]\n", child_ctx->id));
key.type = HASH_KEY_ULONG;
key.ul = child_ctx->id;
- hash_delete(child_ctx->auth_ctx->request_table, &key);
+ hret = hash_delete(child_ctx->auth_ctx->request_table, &key);
+ if (!(hret == HASH_SUCCESS ||
+ hret == HASH_ERROR_KEY_NOT_FOUND)) {
+ DEBUG(1, ("Hash error [%d][%s]\n", hret, hash_error_string(hret)));
+ /* Nothing we can do about this, so just continue */
+ }
return 0;
}