Commit 969b7f25 authored by Trond Myklebust's avatar Trond Myklebust

SUNRPC: Fix a potential race in rpc_pipefs.

Signed-off-by: default avatarTrond Myklebust <Trond.Myklebust@netapp.com>
parent 566dd606
...@@ -70,8 +70,11 @@ rpc_timeout_upcall_queue(void *data) ...@@ -70,8 +70,11 @@ rpc_timeout_upcall_queue(void *data)
struct inode *inode = &rpci->vfs_inode; struct inode *inode = &rpci->vfs_inode;
down(&inode->i_sem); down(&inode->i_sem);
if (rpci->ops == NULL)
goto out;
if (rpci->nreaders == 0 && !list_empty(&rpci->pipe)) if (rpci->nreaders == 0 && !list_empty(&rpci->pipe))
__rpc_purge_upcall(inode, -ETIMEDOUT); __rpc_purge_upcall(inode, -ETIMEDOUT);
out:
up(&inode->i_sem); up(&inode->i_sem);
} }
...@@ -113,8 +116,6 @@ rpc_close_pipes(struct inode *inode) ...@@ -113,8 +116,6 @@ rpc_close_pipes(struct inode *inode)
{ {
struct rpc_inode *rpci = RPC_I(inode); struct rpc_inode *rpci = RPC_I(inode);
cancel_delayed_work(&rpci->queue_timeout);
flush_scheduled_work();
down(&inode->i_sem); down(&inode->i_sem);
if (rpci->ops != NULL) { if (rpci->ops != NULL) {
rpci->nreaders = 0; rpci->nreaders = 0;
...@@ -127,6 +128,8 @@ rpc_close_pipes(struct inode *inode) ...@@ -127,6 +128,8 @@ rpc_close_pipes(struct inode *inode)
} }
rpc_inode_setowner(inode, NULL); rpc_inode_setowner(inode, NULL);
up(&inode->i_sem); up(&inode->i_sem);
cancel_delayed_work(&rpci->queue_timeout);
flush_scheduled_work();
} }
static struct inode * static struct inode *
...@@ -166,7 +169,7 @@ rpc_pipe_open(struct inode *inode, struct file *filp) ...@@ -166,7 +169,7 @@ rpc_pipe_open(struct inode *inode, struct file *filp)
static int static int
rpc_pipe_release(struct inode *inode, struct file *filp) rpc_pipe_release(struct inode *inode, struct file *filp)
{ {
struct rpc_inode *rpci = RPC_I(filp->f_dentry->d_inode); struct rpc_inode *rpci = RPC_I(inode);
struct rpc_pipe_msg *msg; struct rpc_pipe_msg *msg;
down(&inode->i_sem); down(&inode->i_sem);
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment