Skip to content

Commit

Permalink
SUNRPC: Allow rpc_pipefs_ops to have null values for upcall and downcall
Browse files Browse the repository at this point in the history
Also ensure that we use the umode_t type when appropriate...

Signed-off-by: Trond Myklebust <Trond.Myklebust@netapp.com>
  • Loading branch information
Trond Myklebust authored and Trond Myklebust committed Aug 9, 2009
1 parent b693ba4 commit 7364af6
Showing 1 changed file with 11 additions and 4 deletions.
15 changes: 11 additions & 4 deletions net/sunrpc/rpc_pipe.c
Original file line number Diff line number Diff line change
Expand Up @@ -417,7 +417,7 @@ enum {
struct rpc_filelist {
char *name;
const struct file_operations *i_fop;
int mode;
umode_t mode;
};

static struct rpc_filelist files[] = {
Expand Down Expand Up @@ -516,7 +516,7 @@ rpc_release_path(struct nameidata *nd)
}

static struct inode *
rpc_get_inode(struct super_block *sb, int mode)
rpc_get_inode(struct super_block *sb, umode_t mode)
{
struct inode *inode = new_inode(sb);
if (!inode)
Expand Down Expand Up @@ -589,7 +589,8 @@ rpc_populate(struct dentry *parent,
struct inode *inode, *dir = parent->d_inode;
void *private = RPC_I(dir)->private;
struct dentry *dentry;
int mode, i;
umode_t mode;
int i;

mutex_lock(&dir->i_mutex);
for (i = start; i < eof; i++) {
Expand Down Expand Up @@ -783,6 +784,12 @@ struct dentry *rpc_mkpipe(struct dentry *parent, const char *name,
struct dentry *dentry;
struct inode *dir, *inode;
struct rpc_inode *rpci;
umode_t umode = S_IFIFO | S_IRUSR | S_IWUSR;

if (ops->upcall == NULL)
umode &= ~S_IRUGO;
if (ops->downcall == NULL)
umode &= ~S_IWUGO;

dentry = rpc_lookup_create(parent, name, strlen(name), 0);
if (IS_ERR(dentry))
Expand All @@ -799,7 +806,7 @@ struct dentry *rpc_mkpipe(struct dentry *parent, const char *name,
rpci->nkern_readwriters++;
goto out;
}
inode = rpc_get_inode(dir->i_sb, S_IFIFO | S_IRUSR | S_IWUSR);
inode = rpc_get_inode(dir->i_sb, umode);
if (!inode)
goto err_dput;
inode->i_ino = iunique(dir->i_sb, 100);
Expand Down

0 comments on commit 7364af6

Please sign in to comment.