From dbbec1261eae8440726b001df8fe4c06837d6f3d Mon Sep 17 00:00:00 2001 From: Amar Tumballi Date: Mon, 27 Sep 2010 09:26:17 +0000 Subject: distribute: don't check for inode's gfid in _cbk * in dht_lookup_dir_cbk(), at the entry itself we are dereferencing local->inode, which can be NULL in many cases. Hence no need to check/dereference that variable. Just check 'local->gfid'. Signed-off-by: Amar Tumballi Signed-off-by: Vijay Bellur BUG: 1696 () URL: http://bugs.gluster.com/cgi-bin/bugzilla3/show_bug.cgi?id=1696 --- xlators/cluster/dht/src/dht-common.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/xlators/cluster/dht/src/dht-common.c b/xlators/cluster/dht/src/dht-common.c index f57e5538bcf..46685c9e2f4 100644 --- a/xlators/cluster/dht/src/dht-common.c +++ b/xlators/cluster/dht/src/dht-common.c @@ -100,8 +100,7 @@ dht_lookup_dir_cbk (call_frame_t *frame, void *cookie, xlator_t *this, layout = local->layout; - if (!op_ret && uuid_is_null (local->gfid) && - uuid_is_null (local->inode->gfid)) + if (!op_ret && uuid_is_null (local->gfid)) memcpy (local->gfid, stbuf->ia_gfid, 16); LOCK (&frame->lock); -- cgit