From 010da8e41edc510c4c0236a4ec23e9e628faebe7 Mon Sep 17 00:00:00 2001 From: Susant Palai Date: Mon, 30 Jun 2014 14:04:34 -0400 Subject: DHT/permissoin: Let setattr consume stat built from lookup in heal path setattr call post mkdir(selfheal) ends up using the mode bits returned by mkdir,which miss the required suid, sgid and sticky bit. Hence, the fix is to use the mode bits from local->stbuf which was used to create the missing directories. Change-Id: I478708c80e28edc6509b784b0ad83952fc074a5b BUG: 1110262 Signed-off-by: Susant Palai Reviewed-on: http://review.gluster.org/8208 Tested-by: NetBSD Build System Reviewed-by: Shyamsundar Ranganathan Reviewed-by: Raghavendra G Tested-by: Raghavendra G --- xlators/cluster/dht/src/dht-selfheal.c | 2 -- 1 file changed, 2 deletions(-) (limited to 'xlators/cluster/dht/src') diff --git a/xlators/cluster/dht/src/dht-selfheal.c b/xlators/cluster/dht/src/dht-selfheal.c index 6d3d20823cd..2c2e3ae863e 100644 --- a/xlators/cluster/dht/src/dht-selfheal.c +++ b/xlators/cluster/dht/src/dht-selfheal.c @@ -1106,8 +1106,6 @@ dht_selfheal_dir_mkdir_cbk (call_frame_t *frame, void *cookie, xlator_t *this, local->loc.path, gfid ); goto out; } - - dht_iatt_merge (this, &local->stbuf, stbuf, prev->this); dht_iatt_merge (this, &local->preparent, preparent, prev->this); dht_iatt_merge (this, &local->postparent, postparent, prev->this); -- cgit