diff options
| author | Sunny Kumar <sunkumar@redhat.com> | 2018-08-10 02:20:01 +0530 |
|---|---|---|
| committer | Amar Tumballi <amarts@redhat.com> | 2018-08-16 06:13:25 +0000 |
| commit | 0ef74dbd70cce5462b289f6f0cfb05082db00ff5 (patch) | |
| tree | 4681e5ab3d3b4ed72cc478d9392436606e5cb716 /xlators/features/snapview-server/src | |
| parent | e35424da737e50c8ba5cfc12bb2052b2d5ad69ce (diff) | |
uss : fix coverity issues
This patch fixes coverity issuse in snapview-server.c
CID : 1274119, 1325525
Scan details at [1].
[1]. https://scan6.coverity.com/reports.htm#v42401/p10714/fileInstanceId=84476369&defectInstanceId=25631967&mergedDefectId=778645
Change-Id: I825f09eabf84a2262a079c1f920a673727c5792b
updates: bz#789278
Signed-off-by: Sunny Kumar <sunkumar@redhat.com>
Diffstat (limited to 'xlators/features/snapview-server/src')
| -rw-r--r-- | xlators/features/snapview-server/src/snapview-server.c | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/xlators/features/snapview-server/src/snapview-server.c b/xlators/features/snapview-server/src/snapview-server.c index 89684ccac07..9b3f3a54ea7 100644 --- a/xlators/features/snapview-server/src/snapview-server.c +++ b/xlators/features/snapview-server/src/snapview-server.c @@ -901,7 +901,7 @@ svs_getxattr (call_frame_t *frame, xlator_t *this, loc_t *loc, const char *name, } out: - if (op_ret) + if (op_ret && value) GF_FREE (value); STACK_UNWIND_STRICT (getxattr, frame, op_ret, op_errno, dict, NULL); @@ -1337,7 +1337,6 @@ svs_glfs_readdir (xlator_t *this, glfs_fd_t *glfd, gf_dirent_t *entries, break; } dirents = NULL; - ret = -1; } out: |