projects
/
silc.git
/ commitdiff
commit
grep
author
committer
pickaxe
?
search:
re
summary
|
shortlog
|
log
|
commit
| commitdiff |
tree
raw
|
patch
|
inline
| side by side (parent:
fdba394
)
Check for NULL cache argument.
author
Pekka Riikonen
<priikone@silcnet.org>
Wed, 14 Feb 2007 14:46:56 +0000
(14:46 +0000)
committer
Pekka Riikonen
<priikone@silcnet.org>
Wed, 14 Feb 2007 14:46:56 +0000
(14:46 +0000)
lib/silcapputil/silcidcache.c
patch
|
blob
|
history
diff --git
a/lib/silcapputil/silcidcache.c
b/lib/silcapputil/silcidcache.c
index 52ddcef7fdb05151f362f620f79e5851333bdb25..1c23b290236100a20581bf3617f57aeaa652bd26 100644
(file)
--- a/
lib/silcapputil/silcidcache.c
+++ b/
lib/silcapputil/silcidcache.c
@@
-123,7
+123,7
@@
silc_idcache_add(SilcIDCache cache, char *name, void *id, void *context)
{
SilcIDCacheEntry c;
{
SilcIDCacheEntry c;
- if (!id)
+ if (!
cache || !
id)
return NULL;
/* Allocate new cache entry */
return NULL;
/* Allocate new cache entry */
@@
-174,6
+174,9
@@
SilcBool silc_idcache_del(SilcIDCache cache, SilcIDCacheEntry entry,
{
SilcBool ret = FALSE;
{
SilcBool ret = FALSE;
+ if (!cache)
+ return FALSE;
+
SILC_LOG_DEBUG(("Deleting cache entry %p", entry));
if (entry->name)
SILC_LOG_DEBUG(("Deleting cache entry %p", entry));
if (entry->name)
@@
-199,6
+202,9
@@
SilcBool silc_idcache_del_by_id(SilcIDCache cache, void *id,
{
SilcIDCacheEntry c;
{
SilcIDCacheEntry c;
+ if (!cache)
+ return FALSE;
+
if (!silc_hash_table_find(cache->id_table, id, NULL, (void **)&c))
return FALSE;
if (!silc_hash_table_find(cache->id_table, id, NULL, (void **)&c))
return FALSE;
@@
-212,6
+218,9
@@
SilcBool silc_idcache_del_by_context(SilcIDCache cache, void *context,
{
SilcIDCacheEntry c;
{
SilcIDCacheEntry c;
+ if (!cache)
+ return FALSE;
+
if (!silc_hash_table_find(cache->context_table, context, NULL, (void **)&c))
return FALSE;
if (!silc_hash_table_find(cache->context_table, context, NULL, (void **)&c))
return FALSE;
@@
-224,6
+233,9
@@
SilcBool silc_idcache_update(SilcIDCache cache, SilcIDCacheEntry entry,
void *new_id, char *new_name,
SilcBool free_old_name)
{
void *new_id, char *new_name,
SilcBool free_old_name)
{
+ if (!cache)
+ return FALSE;
+
if (new_id) {
if (!silc_hash_table_del_by_context(cache->id_table, entry->id, entry))
return FALSE;
if (new_id) {
if (!silc_hash_table_del_by_context(cache->id_table, entry->id, entry))
return FALSE;
@@
-264,6
+276,9
@@
SilcBool silc_idcache_update_by_context(SilcIDCache cache, void *context,
{
SilcIDCacheEntry c;
{
SilcIDCacheEntry c;
+ if (!cache)
+ return FALSE;
+
if (!silc_hash_table_find(cache->context_table, context, NULL, (void **)&c))
return FALSE;
if (!silc_hash_table_find(cache->context_table, context, NULL, (void **)&c))
return FALSE;
@@
-274,7
+289,7
@@
SilcBool silc_idcache_update_by_context(SilcIDCache cache, void *context,
SilcBool silc_idcache_get_all(SilcIDCache cache, SilcList *ret_list)
{
SilcBool silc_idcache_get_all(SilcIDCache cache, SilcList *ret_list)
{
- if (!ret_list)
+ if (!
cache || !
ret_list)
return FALSE;
if (!silc_hash_table_count(cache->id_table))
return FALSE;
if (!silc_hash_table_count(cache->id_table))
@@
-295,7
+310,7
@@
SilcBool silc_idcache_get_all(SilcIDCache cache, SilcList *ret_list)
SilcBool silc_idcache_find_by_id(SilcIDCache cache, void *id,
SilcList *ret_list)
{
SilcBool silc_idcache_find_by_id(SilcIDCache cache, void *id,
SilcList *ret_list)
{
- if (!ret_list)
+ if (!
cache || !
ret_list)
return FALSE;
if (!silc_hash_table_count(cache->id_table))
return FALSE;
if (!silc_hash_table_count(cache->id_table))
@@
-316,6
+331,8
@@
SilcBool silc_idcache_find_by_id(SilcIDCache cache, void *id,
SilcBool silc_idcache_find_by_id_one(SilcIDCache cache, void *id,
SilcIDCacheEntry *ret)
{
SilcBool silc_idcache_find_by_id_one(SilcIDCache cache, void *id,
SilcIDCacheEntry *ret)
{
+ if (!cache)
+ return FALSE;
return silc_hash_table_find_ext(cache->id_table, id, NULL, (void *)ret,
NULL, NULL,
silc_hash_id_compare_full,
return silc_hash_table_find_ext(cache->id_table, id, NULL, (void *)ret,
NULL, NULL,
silc_hash_id_compare_full,
@@
-327,6
+344,8
@@
SilcBool silc_idcache_find_by_id_one(SilcIDCache cache, void *id,
SilcBool silc_idcache_find_by_context(SilcIDCache cache, void *context,
SilcIDCacheEntry *ret)
{
SilcBool silc_idcache_find_by_context(SilcIDCache cache, void *context,
SilcIDCacheEntry *ret)
{
+ if (!cache)
+ return FALSE;
return silc_hash_table_find(cache->context_table, context, NULL,
(void *)ret);
}
return silc_hash_table_find(cache->context_table, context, NULL,
(void *)ret);
}
@@
-336,7
+355,7
@@
SilcBool silc_idcache_find_by_context(SilcIDCache cache, void *context,
SilcBool silc_idcache_find_by_name(SilcIDCache cache, char *name,
SilcList *ret_list)
{
SilcBool silc_idcache_find_by_name(SilcIDCache cache, char *name,
SilcList *ret_list)
{
- if (!ret_list)
+ if (!
cache || !
ret_list)
return FALSE;
if (!silc_hash_table_count(cache->name_table))
return FALSE;
if (!silc_hash_table_count(cache->name_table))
@@
-357,5
+376,7
@@
SilcBool silc_idcache_find_by_name(SilcIDCache cache, char *name,
SilcBool silc_idcache_find_by_name_one(SilcIDCache cache, char *name,
SilcIDCacheEntry *ret)
{
SilcBool silc_idcache_find_by_name_one(SilcIDCache cache, char *name,
SilcIDCacheEntry *ret)
{
+ if (!cache)
+ return FALSE;
return silc_hash_table_find(cache->name_table, name, NULL, (void *)ret);
}
return silc_hash_table_find(cache->name_table, name, NULL, (void *)ret);
}