summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorNiels de Vos <ndevos@redhat.com>2017-01-13 16:05:02 +0100
committerJeff Darcy <jeff@pl.atyp.us>2017-05-08 13:30:02 +0000
commit52c28c0c04722a9ffaa7c39c49ffebdf0a5c75e1 (patch)
tree2c396445405c05f4af28da9b3df89ba8f3fc9f15
parent68b4db2ac37af3283580b74eef5d405c9370b0c0 (diff)
nfs/nlm: unref rpc-client after nlm4svc_send_granted()
nlm4svc_send_granted() uses the rpc_clnt by getting it from the call-state structure. It is safer to unref the rpc_clnt after the function is done with it. Change-Id: I7cb7c4297801463d21259c58b50d7df7c57aec5e BUG: 1381970 Signed-off-by: Niels de Vos <ndevos@redhat.com> Reviewed-on: https://review.gluster.org/17187 Smoke: Gluster Build System <jenkins@build.gluster.org> NetBSD-regression: NetBSD Build System <jenkins@build.gluster.org> CentOS-regression: Gluster Build System <jenkins@build.gluster.org> Reviewed-by: soumya k <skoduri@redhat.com> Reviewed-by: Jeff Darcy <jeff@pl.atyp.us>
-rw-r--r--xlators/nfs/server/src/nlm4.c2
1 files changed, 1 insertions, 1 deletions
diff --git a/xlators/nfs/server/src/nlm4.c b/xlators/nfs/server/src/nlm4.c
index 8db149cece5..de914353e64 100644
--- a/xlators/nfs/server/src/nlm4.c
+++ b/xlators/nfs/server/src/nlm4.c
@@ -930,8 +930,8 @@ nlm_rpcclnt_notify (struct rpc_clnt *rpc_clnt, void *mydata,
"rpc clnt");
goto err;
}
- rpc_clnt_unref (rpc_clnt);
nlm4svc_send_granted (cs);
+ rpc_clnt_unref (rpc_clnt);
break;