knfsd: clean up EX_RDONLY
Share a little common code, reverse the arguments for consistency, drop the unnecessary "inline", and lowercase the name. Signed-off-by: "J. Bruce Fields" <bfields@citi.umich.edu> Acked-by: Neil Brown <neilb@suse.de> Signed-off-by: Andrew Morton <akpm@linux-foundation.org> Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
This commit is contained in:
committed by
Linus Torvalds
parent
e22841c637
commit
c7d51402d2
@@ -1797,16 +1797,9 @@ nfsd_statfs(struct svc_rqst *rqstp, struct svc_fh *fhp, struct kstatfs *stat)
|
||||
return err;
|
||||
}
|
||||
|
||||
static inline int EX_RDONLY(struct svc_export *exp, struct svc_rqst *rqstp)
|
||||
static int exp_rdonly(struct svc_rqst *rqstp, struct svc_export *exp)
|
||||
{
|
||||
struct exp_flavor_info *f;
|
||||
struct exp_flavor_info *end = exp->ex_flavors + exp->ex_nflavors;
|
||||
|
||||
for (f = exp->ex_flavors; f < end; f++) {
|
||||
if (f->pseudoflavor == rqstp->rq_flavor)
|
||||
return f->flags & NFSEXP_READONLY;
|
||||
}
|
||||
return exp->ex_flags & NFSEXP_READONLY;
|
||||
return nfsexp_flags(rqstp, exp) & NFSEXP_READONLY;
|
||||
}
|
||||
|
||||
/*
|
||||
@@ -1845,7 +1838,7 @@ nfsd_permission(struct svc_rqst *rqstp, struct svc_export *exp,
|
||||
*/
|
||||
if (!(acc & MAY_LOCAL_ACCESS))
|
||||
if (acc & (MAY_WRITE | MAY_SATTR | MAY_TRUNC)) {
|
||||
if (EX_RDONLY(exp, rqstp) || IS_RDONLY(inode))
|
||||
if (exp_rdonly(rqstp, exp) || IS_RDONLY(inode))
|
||||
return nfserr_rofs;
|
||||
if (/* (acc & MAY_WRITE) && */ IS_IMMUTABLE(inode))
|
||||
return nfserr_perm;
|
||||
|
Reference in New Issue
Block a user