lkml.org 
[lkml]   [2007]   [Jun]   [26]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    Patch in this message
    /
    Date
    From
    Subject[AppArmor 05/44] Add struct vfsmount parameter to vfs_mkdir()
    The vfsmount will be passed down to the LSM hook so that LSMs can compute
    pathnames.

    Signed-off-by: Tony Jones <tonyj@suse.de>
    Signed-off-by: Andreas Gruenbacher <agruen@suse.de>
    Signed-off-by: John Johansen <jjohansen@suse.de>

    ---
    fs/ecryptfs/inode.c | 5 ++++-
    fs/namei.c | 5 +++--
    fs/nfsd/nfs4recover.c | 3 ++-
    fs/nfsd/vfs.c | 8 +++++---
    include/linux/fs.h | 2 +-
    5 files changed, 15 insertions(+), 8 deletions(-)

    --- a/fs/ecryptfs/inode.c
    +++ b/fs/ecryptfs/inode.c
    @@ -509,11 +509,14 @@ static int ecryptfs_mkdir(struct inode *
    {
    int rc;
    struct dentry *lower_dentry;
    + struct vfsmount *lower_mnt;
    struct dentry *lower_dir_dentry;

    lower_dentry = ecryptfs_dentry_to_lower(dentry);
    + lower_mnt = ecryptfs_dentry_to_lower_mnt(dentry);
    lower_dir_dentry = lock_parent(lower_dentry);
    - rc = vfs_mkdir(lower_dir_dentry->d_inode, lower_dentry, mode);
    + rc = vfs_mkdir(lower_dir_dentry->d_inode, lower_dentry, lower_mnt,
    + mode);
    if (rc || !lower_dentry->d_inode)
    goto out;
    rc = ecryptfs_interpose(lower_dentry, dentry, dir->i_sb, 0);
    --- a/fs/namei.c
    +++ b/fs/namei.c
    @@ -1934,7 +1934,8 @@ asmlinkage long sys_mknod(const char __u
    return sys_mknodat(AT_FDCWD, filename, mode, dev);
    }

    -int vfs_mkdir(struct inode *dir, struct dentry *dentry, int mode)
    +int vfs_mkdir(struct inode *dir, struct dentry *dentry, struct vfsmount *mnt,
    + int mode)
    {
    int error = may_create(dir, dentry, NULL);

    @@ -1978,7 +1979,7 @@ asmlinkage long sys_mkdirat(int dfd, con

    if (!IS_POSIXACL(nd.dentry->d_inode))
    mode &= ~current->fs->umask;
    - error = vfs_mkdir(nd.dentry->d_inode, dentry, mode);
    + error = vfs_mkdir(nd.dentry->d_inode, dentry, nd.mnt, mode);
    dput(dentry);
    out_unlock:
    mutex_unlock(&nd.dentry->d_inode->i_mutex);
    --- a/fs/nfsd/nfs4recover.c
    +++ b/fs/nfsd/nfs4recover.c
    @@ -156,7 +156,8 @@ nfsd4_create_clid_dir(struct nfs4_client
    dprintk("NFSD: nfsd4_create_clid_dir: DIRECTORY EXISTS\n");
    goto out_put;
    }
    - status = vfs_mkdir(rec_dir.dentry->d_inode, dentry, S_IRWXU);
    + status = vfs_mkdir(rec_dir.dentry->d_inode, dentry, rec_dir.mnt,
    + S_IRWXU);
    out_put:
    dput(dentry);
    out_unlock:
    --- a/fs/nfsd/vfs.c
    +++ b/fs/nfsd/vfs.c
    @@ -1115,6 +1115,7 @@ nfsd_create(struct svc_rqst *rqstp, stru
    int type, dev_t rdev, struct svc_fh *resfhp)
    {
    struct dentry *dentry, *dchild = NULL;
    + struct svc_export *exp;
    struct inode *dirp;
    __be32 err;
    int host_err;
    @@ -1131,6 +1132,7 @@ nfsd_create(struct svc_rqst *rqstp, stru
    goto out;

    dentry = fhp->fh_dentry;
    + exp = fhp->fh_export;
    dirp = dentry->d_inode;

    err = nfserr_notdir;
    @@ -1147,7 +1149,7 @@ nfsd_create(struct svc_rqst *rqstp, stru
    host_err = PTR_ERR(dchild);
    if (IS_ERR(dchild))
    goto out_nfserr;
    - err = fh_compose(resfhp, fhp->fh_export, dchild, fhp);
    + err = fh_compose(resfhp, exp, dchild, fhp);
    if (err)
    goto out;
    } else {
    @@ -1186,7 +1188,7 @@ nfsd_create(struct svc_rqst *rqstp, stru
    host_err = vfs_create(dirp, dchild, iap->ia_mode, NULL);
    break;
    case S_IFDIR:
    - host_err = vfs_mkdir(dirp, dchild, iap->ia_mode);
    + host_err = vfs_mkdir(dirp, dchild, exp->ex_mnt, iap->ia_mode);
    break;
    case S_IFCHR:
    case S_IFBLK:
    @@ -1201,7 +1203,7 @@ nfsd_create(struct svc_rqst *rqstp, stru
    if (host_err < 0)
    goto out_nfserr;

    - if (EX_ISSYNC(fhp->fh_export)) {
    + if (EX_ISSYNC(exp)) {
    err = nfserrno(nfsd_sync_dir(dentry));
    write_inode_now(dchild->d_inode, 1);
    }
    --- a/include/linux/fs.h
    +++ b/include/linux/fs.h
    @@ -993,7 +993,7 @@ extern void unlock_super(struct super_bl
    */
    extern int vfs_permission(struct nameidata *, int);
    extern int vfs_create(struct inode *, struct dentry *, int, struct nameidata *);
    -extern int vfs_mkdir(struct inode *, struct dentry *, int);
    +extern int vfs_mkdir(struct inode *, struct dentry *, struct vfsmount *, int);
    extern int vfs_mknod(struct inode *, struct dentry *, int, dev_t);
    extern int vfs_symlink(struct inode *, struct dentry *, const char *, int);
    extern int vfs_link(struct dentry *, struct inode *, struct dentry *);
    --
    -
    To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
    the body of a message to majordomo@vger.kernel.org
    More majordomo info at http://vger.kernel.org/majordomo-info.html
    Please read the FAQ at http://www.tux.org/lkml/

    \
     
     \ /
      Last update: 2007-06-27 01:15    [W:0.025 / U:60.640 seconds]
    ©2003-2016 Jasper Spaans. hosted at Digital OceanAdvertise on this site