diff options
author | Niels de Vos <ndevos@redhat.com> | 2017-01-13 16:05:02 +0100 |
---|---|---|
committer | Jeff Darcy <jeff@pl.atyp.us> | 2017-05-08 13:30:02 +0000 |
commit | 52c28c0c04722a9ffaa7c39c49ffebdf0a5c75e1 (patch) | |
tree | 2c396445405c05f4af28da9b3df89ba8f3fc9f15 | |
parent | 68b4db2ac37af3283580b74eef5d405c9370b0c0 (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.c | 2 |
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; |