lkml.org 
[lkml]   [2020]   [Oct]   [27]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    Patch in this message
    /
    From
    Subject[PATCH 5.8 065/633] SMB3.1.1: Fix ids returned in POSIX query dir
    Date
    From: Steve French <stfrench@microsoft.com>

    commit 9934430e2178d5164eb1ac91a9b092f9e7e64745 upstream.

    We were setting the uid/gid to the default in each dir entry
    in the parsing of the POSIX query dir response, rather
    than attempting to map the user and group SIDs returned by
    the server to well known SIDs (or upcall if not found).

    CC: Stable <stable@vger.kernel.org>
    Reviewed-by: Aurelien Aptel <aaptel@suse.com>
    Signed-off-by: Steve French <stfrench@microsoft.com>
    Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>

    ---
    fs/cifs/cifsacl.c | 5 +++--
    fs/cifs/cifsproto.h | 2 ++
    fs/cifs/readdir.c | 5 ++---
    3 files changed, 7 insertions(+), 5 deletions(-)

    --- a/fs/cifs/cifsacl.c
    +++ b/fs/cifs/cifsacl.c
    @@ -338,7 +338,7 @@ invalidate_key:
    goto out_key_put;
    }

    -static int
    +int
    sid_to_id(struct cifs_sb_info *cifs_sb, struct cifs_sid *psid,
    struct cifs_fattr *fattr, uint sidtype)
    {
    @@ -359,7 +359,8 @@ sid_to_id(struct cifs_sb_info *cifs_sb,
    return -EIO;
    }

    - if (cifs_sb->mnt_cifs_flags & CIFS_MOUNT_UID_FROM_ACL) {
    + if ((cifs_sb->mnt_cifs_flags & CIFS_MOUNT_UID_FROM_ACL) ||
    + (cifs_sb_master_tcon(cifs_sb)->posix_extensions)) {
    uint32_t unix_id;
    bool is_group;

    --- a/fs/cifs/cifsproto.h
    +++ b/fs/cifs/cifsproto.h
    @@ -208,6 +208,8 @@ extern int cifs_set_file_info(struct ino
    extern int cifs_rename_pending_delete(const char *full_path,
    struct dentry *dentry,
    const unsigned int xid);
    +extern int sid_to_id(struct cifs_sb_info *cifs_sb, struct cifs_sid *psid,
    + struct cifs_fattr *fattr, uint sidtype);
    extern int cifs_acl_to_fattr(struct cifs_sb_info *cifs_sb,
    struct cifs_fattr *fattr, struct inode *inode,
    bool get_mode_from_special_sid,
    --- a/fs/cifs/readdir.c
    +++ b/fs/cifs/readdir.c
    @@ -267,9 +267,8 @@ cifs_posix_to_fattr(struct cifs_fattr *f
    if (reparse_file_needs_reval(fattr))
    fattr->cf_flags |= CIFS_FATTR_NEED_REVAL;

    - /* TODO map SIDs */
    - fattr->cf_uid = cifs_sb->mnt_uid;
    - fattr->cf_gid = cifs_sb->mnt_gid;
    + sid_to_id(cifs_sb, &parsed.owner, fattr, SIDOWNER);
    + sid_to_id(cifs_sb, &parsed.group, fattr, SIDGROUP);
    }

    static void __dir_info_to_fattr(struct cifs_fattr *fattr, const void *info)

    \
     
     \ /
      Last update: 2020-10-27 16:10    [W:2.940 / U:0.348 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site