Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit d3d1fce1 authored by Jeff Layton's avatar Jeff Layton Committed by Steve French
Browse files

cifs: don't override the uid/gid in getattr when cifsacl is enabled 



If we're using cifsacl, then we don't want to override the uid/gid with
the current uid/gid, since that would prevent you from being able to
upcall for this info.

Reviewed-by: default avatarShirish Pargaonkar <shirishpargaonkar@gmail.com>
Signed-off-by: default avatarJeff Layton <jlayton@redhat.com>
Signed-off-by: default avatarSteve French <smfrench@gmail.com>
parent b1a6dc21

fs/cifs/inode.c

+4 −3
Original line number Diff line number Diff line
@@ -1791,11 +1791,12 @@ int cifs_getattr(struct vfsmount *mnt, struct dentry *dentry, 
	stat->ino = CIFS_I(inode)->uniqueid;



	/*

	 * If on a multiuser mount without unix extensions, and the admin hasn't

	 * overridden them, set the ownership to the fsuid/fsgid of the current

	 * process.

	 * If on a multiuser mount without unix extensions or cifsacl being

	 * enabled, and the admin hasn't overridden them, set the ownership

	 * to the fsuid/fsgid of the current process.

	 */

	if ((cifs_sb->mnt_cifs_flags & CIFS_MOUNT_MULTIUSER) &&

	    !(cifs_sb->mnt_cifs_flags & CIFS_MOUNT_CIFS_ACL) &&

	    !tcon->unix_ext) {

		if (!(cifs_sb->mnt_cifs_flags & CIFS_MOUNT_OVERR_UID))

			stat->uid = current_fsuid();