From 2a2f1130689d388f26b41f5df3e0bf572e0addb5 Mon Sep 17 00:00:00 2001 From: Ravishankar N Date: Fri, 29 Aug 2014 10:42:02 +0000 Subject: glusterfs: allow setxattr of keys with null values. Disk based file systems allow to get/set extended attribute key-value pairs where value can be null. Fuse/libgfapi clients must be able to do the same on a gluster volume. Change-Id: Ifc11134cc07f1a3ede43f9d027554dcd10b5c930 BUG: 1135514 Signed-off-by: Ravishankar N Reviewed-on: http://review.gluster.org/8567 Tested-by: Gluster Build System Reviewed-by: Pranith Kumar Karampuri Reviewed-by: Jeff Darcy Reviewed-by: Vijay Bellur --- xlators/storage/posix/src/posix-helpers.c | 4 ++-- xlators/storage/posix/src/posix.c | 4 ++-- 2 files changed, 4 insertions(+), 4 deletions(-) (limited to 'xlators/storage/posix') diff --git a/xlators/storage/posix/src/posix-helpers.c b/xlators/storage/posix/src/posix-helpers.c index 5f8984cc8a7..8584a68a0e8 100644 --- a/xlators/storage/posix/src/posix-helpers.c +++ b/xlators/storage/posix/src/posix-helpers.c @@ -130,7 +130,7 @@ _posix_xattr_get_set_from_backend (posix_xattr_filler_t *filler, char *key) xattr_size = sys_lgetxattr (filler->real_path, key, NULL, 0); - if (xattr_size > 0) { + if (xattr_size != -1) { value = GF_CALLOC (1, xattr_size + 1, gf_posix_mt_char); if (!value) @@ -138,7 +138,7 @@ _posix_xattr_get_set_from_backend (posix_xattr_filler_t *filler, char *key) xattr_size = sys_lgetxattr (filler->real_path, key, value, xattr_size); - if (xattr_size <= 0) { + if (xattr_size == -1) { gf_log (filler->this->name, GF_LOG_WARNING, "getxattr failed. path: %s, key: %s", filler->real_path, key); diff --git a/xlators/storage/posix/src/posix.c b/xlators/storage/posix/src/posix.c index 090e53f0379..43bd3fc1cfd 100644 --- a/xlators/storage/posix/src/posix.c +++ b/xlators/storage/posix/src/posix.c @@ -3703,7 +3703,7 @@ posix_getxattr (call_frame_t *frame, xlator_t *this, } #endif size = sys_lgetxattr (real_path, key, NULL, 0); - if (size <= 0) { + if (size == -1) { op_errno = errno; if ((op_errno == ENOTSUP) || (op_errno == ENOSYS)) { GF_LOG_OCCASIONALLY (gf_posix_xattr_enotsup_log, @@ -3928,7 +3928,7 @@ posix_fgetxattr (call_frame_t *frame, xlator_t *this, } #endif size = sys_fgetxattr (_fd, key, NULL, 0); - if (size <= 0) { + if (size == -1) { op_errno = errno; gf_log (this->name, ((errno == ENODATA || errno == ENOATTR) ? -- cgit