summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorJiffin Tony Thottan <jthottan@redhat.com>2016-05-24 18:52:52 +0530
committerNiels de Vos <ndevos@redhat.com>2016-05-31 04:44:41 -0700
commit719e9f2228027a85df5c7b7b3e5fb4494bfbf186 (patch)
tree0deb99fb24dc6830c5c4fcf621cbd296d75668b3
parentd6442d8f4ac6d892cf5adc55ea5c34dcb01e1f26 (diff)
gfapi/upcall : remove unused variable ret in glfs_h_find_handle
The variable ret is declared in glfs_h_find_handle(), but never used. So while compiling following warning message will be printed. glfs-handleops.c: In function ‘glfs_h_find_handle’: glfs-handleops.c:1813:29: warning: variable ‘ret’ set but not used [-Wunused-but-set-variable] int ret = -1; ^ Change-Id: I599b7a4e87184aacd6c937e946defd4e270509c4 BUG: 1341007 Signed-off-by: Jiffin Tony Thottan <jthottan@redhat.com> Reviewed-on: http://review.gluster.org/14524 Smoke: Gluster Build System <jenkins@build.gluster.com> NetBSD-regression: NetBSD Build System <jenkins@build.gluster.org> CentOS-regression: Gluster Build System <jenkins@build.gluster.com> Reviewed-by: Niels de Vos <ndevos@redhat.com>
-rw-r--r--api/src/glfs-handleops.c2
1 files changed, 0 insertions, 2 deletions
diff --git a/api/src/glfs-handleops.c b/api/src/glfs-handleops.c
index af58691bd8b..9e41fccd28e 100644
--- a/api/src/glfs-handleops.c
+++ b/api/src/glfs-handleops.c
@@ -1810,7 +1810,6 @@ GFAPI_SYMVER_PUBLIC_DEFAULT(glfs_h_rename, 3.4.2);
struct glfs_object *
glfs_h_find_handle (struct glfs *fs, unsigned char *handle, int len)
{
- int ret = -1;
inode_t *newinode = NULL;
xlator_t *subvol = NULL;
struct glfs_object *object = NULL;
@@ -1847,7 +1846,6 @@ glfs_h_find_handle (struct glfs *fs, unsigned char *handle, int len)
glfs_mt_glfs_object_t);
if (object == NULL) {
errno = ENOMEM;
- ret = -1;
goto out;
}