diff options
author | Amir Goldstein <amir73il@gmail.com> | 2019-01-22 00:01:39 -0500 |
---|---|---|
committer | Miklos Szeredi <mszeredi@redhat.com> | 2019-05-06 07:54:50 -0400 |
commit | 3428030da004a1128cbdcf93dc03e16f184d845b (patch) | |
tree | fa8bdd8c7fa12a50499519fe18123e4b271263d1 /lib/netdev-notifier-error-inject.c | |
parent | e93c9c99a629c61837d5a7fc2120cd2b6c70dbdd (diff) |
ovl: fix missing upper fs freeze protection on copy up for ioctl
Generalize the helper ovl_open_maybe_copy_up() and use it to copy up file
with data before FS_IOC_SETFLAGS ioctl.
The FS_IOC_SETFLAGS ioctl is a bit of an odd ball in vfs, which probably
caused the confusion. File may be open O_RDONLY, but ioctl modifies the
file. VFS does not call mnt_want_write_file() nor lock inode mutex, but
fs-specific code for FS_IOC_SETFLAGS does. So ovl_ioctl() calls
mnt_want_write_file() for the overlay file, and fs-specific code calls
mnt_want_write_file() for upper fs file, but there was no call for
ovl_want_write() for copy up duration which prevents overlayfs from copying
up on a frozen upper fs.
Fixes: dab5ca8fd9dd ("ovl: add lsattr/chattr support")
Cc: <stable@vger.kernel.org> # v4.19
Signed-off-by: Amir Goldstein <amir73il@gmail.com>
Acked-by: Vivek Goyal <vgoyal@redhat.com>
Signed-off-by: Miklos Szeredi <mszeredi@redhat.com>
Diffstat (limited to 'lib/netdev-notifier-error-inject.c')
0 files changed, 0 insertions, 0 deletions