xfs: fix sgid inheritance for subdirectories inheriting default acls [V3]
authorCarlos Maiolino <cmaiolino@redhat.com>
Fri, 21 Jun 2013 17:45:53 +0000 (14:45 -0300)
committerBen Myers <bpm@sgi.com>
Wed, 10 Jul 2013 15:21:51 +0000 (10:21 -0500)
commit42c49d7f249c2487f36d3314753d5d8ebcee8249
tree6c0863677aa03d43d5a5c3e8cb5a53c4b1653b08
parentb0a9dab78aee2a479d7c226e6939d553967e4024
xfs: fix sgid inheritance for subdirectories inheriting default acls [V3]

XFS removes sgid bits of subdirectories under a directory containing a default
acl.

When a default acl is set, it implies xfs to call xfs_setattr_nonsize() in its
code path. Such function is shared among mkdir and chmod system calls, and
does some checks unneeded by mkdir (calling inode_change_ok()). Such checks
remove sgid bit from the inode after it has been granted.

With this patch, we extend the meaning of XFS_ATTR_NOACL flag to avoid these
checks when acls are being inherited (thanks hch).

Also, xfs_setattr_mode, doesn't need to re-check for group id and capabilities
permissions, this only implies in another try to remove sgid bit from the
directories. Such check is already done either on inode_change_ok() or
xfs_setattr_nonsize().

Changelog:

V2: Extends the meaning of XFS_ATTR_NOACL instead of wrap the tests into another
    function

V3: Remove S_ISDIR check in xfs_setattr_nonsize() from the patch

Signed-off-by: Carlos Maiolino <cmaiolino@redhat.com>
Reviewed-by: Ben Myers <bpm@sgi.com>
Signed-off-by: Ben Myers <bpm@sgi.com>
fs/xfs/xfs_iops.c