From 09ed0531bd882726e2965d2274d5c9171998f9c8 Mon Sep 17 00:00:00 2001 From: Amar Tumballi Date: Mon, 25 Jul 2011 14:45:12 +0530 Subject: storage/posix: handle dictionary being NULL in a function Change-Id: Ib9cac6ed1635203802f089986f8acb1ce416265d BUG: 3215 Reviewed-on: http://review.gluster.com/97 Tested-by: Gluster Build System Reviewed-by: Anand Avati --- xlators/cluster/dht/src/dht-selfheal.c | 2 +- xlators/storage/posix/src/posix.c | 3 +++ 2 files changed, 4 insertions(+), 1 deletion(-) diff --git a/xlators/cluster/dht/src/dht-selfheal.c b/xlators/cluster/dht/src/dht-selfheal.c index ddd043dc8..d9f2b8b27 100644 --- a/xlators/cluster/dht/src/dht-selfheal.c +++ b/xlators/cluster/dht/src/dht-selfheal.c @@ -346,7 +346,7 @@ dht_selfheal_dir_mkdir (call_frame_t *frame, loc_t *loc, } if (!dict) - gf_log (this->name, GF_LOG_DEBUG, + gf_log (this->name, GF_LOG_WARNING, "dict is NULL, need to make sure gfid's are same"); for (i = 0; i < layout->cnt; i++) { diff --git a/xlators/storage/posix/src/posix.c b/xlators/storage/posix/src/posix.c index b3ed23a45..785d16ec4 100644 --- a/xlators/storage/posix/src/posix.c +++ b/xlators/storage/posix/src/posix.c @@ -516,6 +516,9 @@ posix_entry_create_xattr_set (xlator_t *this, const char *path, data_pair_t *trav = NULL; int ret = -1; + if (!dict) + goto out; + trav = dict->members_list; while (trav) { if (!strcmp (GFID_XATTR_KEY, trav->key) || -- cgit