kernel.git/fs/9p/acl.c, branch linux-3.10.y

posix_acl: Clear SGID bit when setting file permissions

2017-06-07T22:46:47Z

commit 073931017b49d9458aa351605b43a7e34598caef upstream. When file permissions are modified via chmod(2) and the user is not in the owning group or capable of CAP_FSETID, the setgid bit is cleared in inode_change_ok(). Setting a POSIX ACL via setxattr(2) sets the file permissions as well as the new ACL, but doesn't clear the setgid bit in a similar way; this allows to bypass the check in chmod(2). Fix that. Reviewed-by: Christoph Hellwig Reviewed-by: Jeff Layton Signed-off-by: Jan Kara Signed-off-by: Andreas Gruenbacher [wt: dropped hfsplus changes : no xattr in 3.10] Signed-off-by: Willy Tarreau

9p: switch v9fs_set_create_acl() to inode+fid, do it before d_instantiate()

2013-02-26T07:46:07Z

Signed-off-by: Al Viro

9p: split dropping the acls from v9fs_set_create_acl()

2013-02-26T07:46:06Z

Signed-off-by: Al Viro

9p: switch v9fs_acl_chmod() from dentry to inode+fid

2013-02-26T07:46:06Z

caller has both, might as well pass them explicitly. Signed-off-by: Al Viro

9p: switch v9fs_set_acl() from dentry to fid

2013-02-26T07:46:06Z

Signed-off-by: Al Viro

9p: lift the call of set_cached_acl() into the callers of v9fs_set_acl()

2013-02-26T07:46:05Z

Signed-off-by: Al Viro

userns: Pass a userns parameter into posix_acl_to_xattr and posix_acl_from_xattr

2012-09-18T08:01:35Z

- Pass the user namespace the uid and gid values in the xattr are stored in into posix_acl_from_xattr. - Pass the user namespace kuid and kgid values should be converted into when storing uid and gid values in an xattr in posix_acl_to_xattr. - Modify all callers of posix_acl_from_xattr and posix_acl_to_xattr to pass in &init_user_ns. In the short term this change is not strictly needed but it makes the code clearer. In the longer term this change is necessary to be able to mount filesystems outside of the initial user namespace that natively store posix acls in the linux xattr format. Cc: Theodore Tso Cc: Andrew Morton Cc: Andreas Dilger Cc: Jan Kara Cc: Al Viro Signed-off-by: "Eric W. Biederman"

switch posix_acl_equiv_mode() to umode_t *

2011-08-01T06:10:06Z

... so that &inode->i_mode could be passed to it Signed-off-by: Al Viro

switch posix_acl_create() to umode_t *

2011-08-01T06:09:42Z

so we can pass &inode->i_mode to it Signed-off-by: Al Viro

p9: avoid unused variable warning

2011-07-26T06:43:53Z

Commit 4e34e719e457 ("fs: take the ACL checks to common code") removed the use of the 'acl' variable in v9fs_iop_get_acl(), but left the variable definition around. Remove it to get rid of the warning: fs/9p/acl.c: In function ‘v9fs_iop_get_acl’: fs/9p/acl.c:101:20: warning: unused variable ‘acl’ Signed-off-by: Linus Torvalds