From 8aee74f25cfa9dc676e116e9882723254b0509a9 Mon Sep 17 00:00:00 2001
From: Mohammed Rafi KC <rkavunga@redhat.com>
Date: Wed, 19 Oct 2016 15:20:45 +0530
Subject: 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>
---
 glusterfsd/src/glusterfsd-mgmt.c | 2 ++
 1 file changed, 2 insertions(+)

(limited to 'glusterfsd')

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);
-- 
cgit