Re: [PATCH v1 2/2] idmapped-mounts: Add umask before test setgid_create

[Date Prev] [Date Next] [Thread Prev] [Thread Next] [Date Index] [Thread Index]



On Thu, Mar 31, 2022 at 05:28:22PM +0800, Yang Xu wrote:
> Since stipping S_SIGID should check S_IXGRP, so umask it to check whether
> works well.
> 
> Signed-off-by: Yang Xu <xuyang2018.jy@xxxxxxxxxxx>
> ---

(Sidenote: I really need to rename the test binary to something other
than idmapped-mounts.c as this tests a lot of generic vfs stuff that has
nothing to do with them.)

Tested-by: Christian Brauner (Microsoft) <brauner@xxxxxxxxxx>
Reviewed-by: Christian Brauner (Microsoft) <brauner@xxxxxxxxxx>

> If we enable acl on parent directory, then umask is useless, maybe we
> also add setfacl on parent directory because we may change the order
> about strip S_ISGID and posix_acl setup. Any idea?

If acls figure into this then this should probably be a new test or
subtest.



[Index of Archives]     [Linux Filesystems Development]     [Linux NFS]     [Linux NILFS]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux