From d15dedd8c99e84018a50130a8ffe5e971b9f7bd4 Mon Sep 17 00:00:00 2001 From: Pranith Kumar K Date: Thu, 26 Mar 2015 21:08:12 +0530 Subject: xlators/lib: Handle NULL 'name' for marker xattrs Change-Id: I18f00b7e92f483673250821c457d1e8be2eef081 BUG: 1200372 Signed-off-by: Pranith Kumar K Reviewed-on: http://review.gluster.org/10015 Tested-by: Gluster Build System Reviewed-by: Aravinda VK Reviewed-by: Vijay Bellur --- tests/basic/geo-replication/marker-xattrs.t | 2 ++ xlators/lib/src/libxlator.c | 3 +++ 2 files changed, 5 insertions(+) diff --git a/tests/basic/geo-replication/marker-xattrs.t b/tests/basic/geo-replication/marker-xattrs.t index a7dcb50aa7..7061b4532a 100755 --- a/tests/basic/geo-replication/marker-xattrs.t +++ b/tests/basic/geo-replication/marker-xattrs.t @@ -30,6 +30,7 @@ TEST kill_brick $V0 $H0 $B0/${V0}-0 TEST "getfattr -n $xtime $M1 | grep -q ${xtime}=" +TEST getfattr -d -m. -e hex $M1 EXPECT_WITHIN $UMOUNT_TIMEOUT "Y" force_umount $M0 EXPECT_WITHIN $UMOUNT_TIMEOUT "Y" force_umount $M1 @@ -65,6 +66,7 @@ TEST kill_brick $V0 $H0 $B0/${V0}-0 TEST "getfattr -n $xtime $M1 | grep -q ${xtime}=" +TEST getfattr -d -m. -e hex $M1 EXPECT_WITHIN $UMOUNT_TIMEOUT "Y" force_umount $M0 EXPECT_WITHIN $UMOUNT_TIMEOUT "Y" force_umount $M1 diff --git a/xlators/lib/src/libxlator.c b/xlators/lib/src/libxlator.c index 30436131b6..0f703904fa 100644 --- a/xlators/lib/src/libxlator.c +++ b/xlators/lib/src/libxlator.c @@ -458,6 +458,9 @@ cluster_handle_marker_getxattr (call_frame_t *frame, loc_t *loc, if (GF_CLIENT_PID_GSYNCD != frame->root->pid) return -EINVAL; + if (name == NULL) + return -EINVAL; + if (strcmp (GF_XATTR_MARKER_KEY, name) == 0) { type = MARKER_UUID_TYPE; memcpy (gauge, marker_uuid_default_gauge, sizeof (gauge)); -- cgit