diff options
| author | Stephen Gallagher <sgallagh@redhat.com> | 2010-06-09 10:40:49 -0400 |
|---|---|---|
| committer | Stephen Gallagher <sgallagh@redhat.com> | 2010-06-10 10:17:38 -0400 |
| commit | 361e86478ff1cc7ac08981b721f508363291028c (patch) | |
| tree | 9da9b542332bc86d108cc889aea4ab84ded0dd7c /src | |
| parent | 476b2f491e248b182c0f6c3461028b61a8a87ab7 (diff) | |
| download | sssd-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
Diffstat (limited to 'src')
| -rw-r--r-- | src/providers/proxy/proxy.c | 8 |
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; } |