Commit 3e0d9021 authored by Jimmy Durand Wesolowski's avatar Jimmy Durand Wesolowski Committed by Stefan Bader

fs: add the fsnotify call to vfs_iter_write

BugLink: https://bugs.launchpad.net/bugs/1818803

A bug has been discovered when redirecting splice output to regular files
on EXT4 and tmpfs. Other filesystems might be affected.
This commit fixes the issue for stable series kernel, using one of the
change introduced during the rewrite and refactoring of vfs_iter_write in
4.13, specifically in the
commit abbb6589 ("fs: implement vfs_iter_write using do_iter_write").

This issue affects v4.4 and v4.9 stable series of kernels.

Without this fix for v4.4 and v4.9 stable, the following upstream commits
(and their dependencies would need to be backported):
* commit abbb6589 ("fs: implement vfs_iter_write using do_iter_write")
* commit 18e9710e ("fs: implement vfs_iter_read using do_iter_read")
* commit edab5fe3
  ("fs: move more code into do_iter_read/do_iter_write")
* commit 19c73586 ("fs: remove __do_readv_writev")
* commit 26c87fb7 ("fs: remove do_compat_readv_writev")
* commit 251b42a1 ("fs: remove do_readv_writev")

as well as the following dependencies:
* commit bb7462b6
  ("vfs: use helpers for calling f_op->{read,write}_iter()")
* commit 0f78d06a
  ("vfs: pass type instead of fn to do_{loop,iter}_readv_writev()")
* commit 7687a7a4
  ("vfs: extract common parts of {compat_,}do_readv_writev()")

In order to reduce the changes, this commit uses only the part of
commit abbb6589 ("fs: implement vfs_iter_write using do_iter_write")
that fixes the issue.

This issue and the reproducer can be found on
https://bugzilla.kernel.org/show_bug.cgi?id=85381Reported-by: default avatarRichard Li <richardpku@gmail.com>
Reported-by: default avatarChad Miller <millchad@amazon.com>
Reviewed-by: default avatarStefan Nuernberger <snu@amazon.de>
Reviewed-by: default avatarFrank Becker <becke@amazon.de>
Signed-off-by: default avatarJimmy Durand Wesolowski <jdw@amazon.de>
Signed-off-by: default avatarJuerg Haefliger <juergh@canonical.com>
Signed-off-by: default avatarKhalid Elmously <khalid.elmously@canonical.com>
parent 65fcb10a
......@@ -363,8 +363,10 @@ ssize_t vfs_iter_write(struct file *file, struct iov_iter *iter, loff_t *ppos)
iter->type |= WRITE;
ret = file->f_op->write_iter(&kiocb, iter);
BUG_ON(ret == -EIOCBQUEUED);
if (ret > 0)
if (ret > 0) {
*ppos = kiocb.ki_pos;
fsnotify_modify(file);
}
return ret;
}
EXPORT_SYMBOL(vfs_iter_write);
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment