summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorMohammed Rafi KC <rkavunga@redhat.com>2016-10-19 15:20:45 +0530
committerRaghavendra G <rgowdapp@redhat.com>2017-02-27 00:52:50 -0500
commit8aee74f25cfa9dc676e116e9882723254b0509a9 (patch)
tree7e41d057979d6a97976872f39f25fc9362822a5a
parentc1fc1fc9cb5a13e6ddf8c9270deb0c7609333540 (diff)
protocol/mgmt: sending wrong return value during emancipation
A change introduced for moving emancipation after signin process cause this regression, where the second signin always return the size of process xdr. This patch will fix the issue. Change-Id: Ic924c82abe6932a93abe37df1fd2d1285a77ed0a BUG: 1386578 Signed-off-by: Mohammed Rafi KC <rkavunga@redhat.com> Reviewed-on: https://review.gluster.org/15687 CentOS-regression: Gluster Build System <jenkins@build.gluster.org> Reviewed-by: Atin Mukherjee <amukherj@redhat.com> NetBSD-regression: NetBSD Build System <jenkins@build.gluster.org> Smoke: Gluster Build System <jenkins@build.gluster.org> Reviewed-by: Raghavendra G <rgowdapp@redhat.com>
-rw-r--r--glusterfsd/src/glusterfsd-mgmt.c2
1 files changed, 2 insertions, 0 deletions
diff --git a/glusterfsd/src/glusterfsd-mgmt.c b/glusterfsd/src/glusterfsd-mgmt.c
index 8237c23811a..57f8dc0405c 100644
--- a/glusterfsd/src/glusterfsd-mgmt.c
+++ b/glusterfsd/src/glusterfsd-mgmt.c
@@ -2413,6 +2413,8 @@ mgmt_pmap_signin2_cbk (struct rpc_req *req, struct iovec *iov, int count,
ret = -1;
goto out;
}
+
+ ret = 0;
out:
if (need_emancipate)
emancipate (ctx, ret);