ovl: rely on SB_I_NOUMASK
authorChristian Brauner <brauner@kernel.org>
Thu, 12 Oct 2023 15:36:57 +0000 (17:36 +0200)
committerChristian Brauner <brauner@kernel.org>
Thu, 19 Oct 2023 09:03:16 +0000 (11:03 +0200)
commit2bc5e5e8167f2114976f00755b9a0c7f17d6f105
tree86326d71184fdf98bd13f1b7f3ea9105d8bc653a
parente4e8b47a34a432c3f65534d12d5c132b6639da71
ovl: rely on SB_I_NOUMASK

In commit f61b9bb3f838 ("fs: add a new SB_I_NOUMASK flag") we added a
new SB_I_NOUMASK flag that is used by filesystems like NFS to indicate
that umask stripping is never supposed to be done in the vfs independent
of whether or not POSIX ACLs are supported.

Overlayfs falls into the same category as it raises SB_POSIXACL
unconditionally to defer umask application to the upper filesystem.

Now that we have SB_I_NOUMASK use that and make SB_POSIXACL properly
conditional on whether or not the kernel does have support for it. This
will enable use to turn IS_POSIXACL() into nop on kernels that don't
have POSIX ACL support avoding bugs from missed umask stripping.

Link: https://lore.kernel.org/r/20231012-einband-uferpromenade-80541a047a1f@brauner
Acked-by: Amir Goldstein <amir73il@gmail.com>
Signed-off-by: Christian Brauner <brauner@kernel.org>
fs/overlayfs/super.c