]> git.baikalelectronics.ru Git - kernel.git/commit
ntfs: fix acl handling
authorChristian Brauner <brauner@kernel.org>
Wed, 20 Jul 2022 12:32:52 +0000 (14:32 +0200)
committerChristian Brauner (Microsoft) <brauner@kernel.org>
Mon, 22 Aug 2022 10:52:23 +0000 (12:52 +0200)
commitea7ffb66c368e33d2dd74ed50d4aa1c5e258284e
tree451e70daa47b17a87fdb552f2f3cd83350fdd8ef
parent466d88a2ee5eec910db3eee9cd1f0313605b3f5e
ntfs: fix acl handling

While looking at our current POSIX ACL handling in the context of some
overlayfs work I went through a range of other filesystems checking how they
handle them currently and encountered ntfs3.

The posic_acl_{from,to}_xattr() helpers always need to operate on the
filesystem idmapping. Since ntfs3 can only be mounted in the initial user
namespace the relevant idmapping is init_user_ns.

The posix_acl_{from,to}_xattr() helpers are concerned with translating between
the kernel internal struct posix_acl{_entry} and the uapi struct
posix_acl_xattr_{header,entry} and the kernel internal data structure is cached
filesystem wide.

Additional idmappings such as the caller's idmapping or the mount's idmapping
are handled higher up in the VFS. Individual filesystems usually do not need to
concern themselves with these.

The posix_acl_valid() helper is concerned with checking whether the values in
the kernel internal struct posix_acl can be represented in the filesystem's
idmapping. IOW, if they can be written to disk. So this helper too needs to
take the filesystem's idmapping.

Fixes: 8265e412e05b ("fs/ntfs3: Add attrib operations")
Cc: Konstantin Komarov <almaz.alexandrovich@paragon-software.com>
Cc: ntfs3@lists.linux.dev
Signed-off-by: Christian Brauner (Microsoft) <brauner@kernel.org>
fs/ntfs3/xattr.c