Re: [PATCH 5.15.y] Revert "ext4: apply umask if ACL support is disabled"

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

 



On Mon, Feb 10, 2025 at 09:28:01PM +0100, Jesper Nilsson wrote:
> From: Max Kellermann <max.kellermann@xxxxxxxxx>
> 
> This reverts commit 484fd6c1de13b336806a967908a927cc0356e312.  The
> commit caused a regression because now the umask was applied to
> symlinks and the fix is unnecessary because the umask/O_TMPFILE bug
> has been fixed somewhere else already.
> 
> Fixes: https://lore.kernel.org/lkml/28DSITL9912E1.2LSZUVTGTO52Q@xxxxxxxxxxx/
> Signed-off-by: Max Kellermann <max.kellermann@xxxxxxxxx>
> Reviewed-by: Christian Brauner <brauner@xxxxxxxxxx>
> Tested-by: Michael Forney <mforney@xxxxxxxxxxx>
> Link: https://lore.kernel.org/r/20240315142956.2420360-1-max.kellermann@xxxxxxxxx
> Signed-off-by: Theodore Ts'o <tytso@xxxxxxx>
> (cherry picked from commit c77194965dd0dcc26f9c1671d2e74e4eb1248af5)
> 
> ---
> This revert never reached linux-5.15.y, and caused the same regression
> with symbolic links. The original problem is just as the revert states
> fixed and the test program for O_TMPFILE works as expected.

This revert never hit any of the other stable kernels either, should it
go to all of them?

Also, you didn't sign off on this :(

thanks,

greg k-h




[Index of Archives]     [Linux Kernel]     [Kernel Development Newbies]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite Hiking]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux