diff options
| author | Michael Scherer <misc@redhat.com> | 2017-05-03 16:54:46 +0200 |
|---|---|---|
| committer | Jeff Darcy <jeff@pl.atyp.us> | 2017-05-04 15:42:36 +0000 |
| commit | b3229ed1071b8e45b0717abef0adf1c17c9df81c (patch) | |
| tree | 001730aa2016f977e48aa69b215e4229b71f681c | |
| parent | 9f930646eeed12a243d083c91acb59fbe7004074 (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.c | 1 |
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 { |