diff options
author | Ashish Pandey <aspandey@redhat.com> | 2018-09-06 16:15:01 +0530 |
---|---|---|
committer | Amar Tumballi <amarts@redhat.com> | 2018-09-11 04:15:56 +0000 |
commit | ee79077c0d19ddf97caea22cc001b6f4f9b0829c (patch) | |
tree | 95a3861a3659be1c6d1c7d7c06a02d72949f97a9 /xlators/storage | |
parent | 5537a8f050a07d8f1c32138d787331d61695ce38 (diff) |
storage/posix: Fix coverity issue - Unchecked return value
Fixes CID: 1388886
https://scan6.coverity.com/reports.htm#v42607/p10714/fileInstanceId=85287446&defectInstanceId=25997291&mergedDefectId=1388886
Change-Id: Ic4e558bba7e15d213c07bc31affb2e175ace5502
updates: bz#789278
Signed-off-by: Ashish Pandey <aspandey@redhat.com>
Diffstat (limited to 'xlators/storage')
-rw-r--r-- | xlators/storage/posix/src/posix-common.c | 9 |
1 files changed, 8 insertions, 1 deletions
diff --git a/xlators/storage/posix/src/posix-common.c b/xlators/storage/posix/src/posix-common.c index cc9c9ca5c8f..24e990cafe2 100644 --- a/xlators/storage/posix/src/posix-common.c +++ b/xlators/storage/posix/src/posix-common.c @@ -599,7 +599,14 @@ posix_init (xlator_t *this) op_ret = sys_lsetxattr (dir_data->data, "trusted.glusterfs.test", "working", 8, 0); if (op_ret != -1) { - sys_lremovexattr (dir_data->data, "trusted.glusterfs.test"); + ret = sys_lremovexattr (dir_data->data, + "trusted.glusterfs.test"); + if (ret) { + gf_msg (this->name, GF_LOG_DEBUG, errno, + P_MSG_INVALID_OPTION, + "failed to remove xattr: " + "trusted.glusterfs.test"); + } } else { tmp_data = dict_get (this->options, "mandate-attribute"); |