diff options
author | Stanislav Kinsbursky <skinsbursky@parallels.com> | 2012-04-20 10:11:02 -0400 |
---|---|---|
committer | Trond Myklebust <Trond.Myklebust@netapp.com> | 2012-04-27 14:09:59 -0400 |
commit | a4dff1bc492ee4a2184d384ae8b5bcab5859e150 (patch) | |
tree | b6bdb438cfa7c041d6e0a7b42f8c10c213961215 /net/sunrpc/clnt.c | |
parent | 5794d21ef4639f0e33440927bb903f9598c21e92 (diff) |
SUNRPC: skip dead but not buried clients on PipeFS events
These clients can't be safely dereferenced if their counter in 0.
Signed-off-by: Stanislav Kinsbursky <skinsbursky@parallels.com>
Signed-off-by: Trond Myklebust <Trond.Myklebust@netapp.com>
Diffstat (limited to 'net/sunrpc/clnt.c')
-rw-r--r-- | net/sunrpc/clnt.c | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/net/sunrpc/clnt.c b/net/sunrpc/clnt.c index 67972462a543..d10ebc4310f7 100644 --- a/net/sunrpc/clnt.c +++ b/net/sunrpc/clnt.c | |||
@@ -218,7 +218,8 @@ static struct rpc_clnt *rpc_get_client_for_event(struct net *net, int event) | |||
218 | if (((event == RPC_PIPEFS_MOUNT) && clnt->cl_dentry) || | 218 | if (((event == RPC_PIPEFS_MOUNT) && clnt->cl_dentry) || |
219 | ((event == RPC_PIPEFS_UMOUNT) && !clnt->cl_dentry)) | 219 | ((event == RPC_PIPEFS_UMOUNT) && !clnt->cl_dentry)) |
220 | continue; | 220 | continue; |
221 | atomic_inc(&clnt->cl_count); | 221 | if (atomic_inc_not_zero(&clnt->cl_count) == 0) |
222 | continue; | ||
222 | spin_unlock(&sn->rpc_client_lock); | 223 | spin_unlock(&sn->rpc_client_lock); |
223 | return clnt; | 224 | return clnt; |
224 | } | 225 | } |