summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorMichael Scherer <misc@redhat.com>2017-05-03 16:54:46 +0200
committerJeff Darcy <jeff@pl.atyp.us>2017-05-04 15:42:36 +0000
commitb3229ed1071b8e45b0717abef0adf1c17c9df81c (patch)
tree001730aa2016f977e48aa69b215e4229b71f681c
parent9f930646eeed12a243d083c91acb59fbe7004074 (diff)
Remove double free
Since there is already GF_FREE (value) near the out label, this one is uneeded. Found by coverity. Change-Id: Ib4a4e1a6af186e14fc75e7578c79beb58de053ec BUG: 789278 Signed-off-by: Michael Scherer <misc@redhat.com> Reviewed-on: https://review.gluster.org/17165 Smoke: Gluster Build System <jenkins@build.gluster.org> Tested-by: Michael Scherer <misc@fedoraproject.org> NetBSD-regression: NetBSD Build System <jenkins@build.gluster.org> CentOS-regression: Gluster Build System <jenkins@build.gluster.org> Reviewed-by: Amar Tumballi <amarts@redhat.com>
-rw-r--r--xlators/features/snapview-server/src/snapview-server.c1
1 files changed, 0 insertions, 1 deletions
diff --git a/xlators/features/snapview-server/src/snapview-server.c b/xlators/features/snapview-server/src/snapview-server.c
index ad169b2466d..5cee414414b 100644
--- a/xlators/features/snapview-server/src/snapview-server.c
+++ b/xlators/features/snapview-server/src/snapview-server.c
@@ -1009,7 +1009,6 @@ svs_fgetxattr (call_frame_t *frame, xlator_t *this, fd_t *fd, const char *name,
"operation for gfid %s for the key %s "
"failed.",
uuid_utoa (fd->inode->gfid), name);
- GF_FREE (value);
goto out;
}
} else {