CVE-2022-1011: kernel/git/mszeredi/fuse.git - FUSE

In FOPEN_DIRECT_IO mode, fuse_file_write_iter() calls fuse_direct_write_iter(), which normally calls fuse_direct_io(), which then imports the write buffer with fuse_get_user_pages(), which uses iov_iter_get_pages() to grab references to userspace pages instead of actually copying memory. On the filesystem device side, these pages can then either be read to userspace (via fuse_dev_read()), or splice()d over into a pipe using fuse_dev_splice_read() as pipe buffers with &nosteal_pipe_buf_ops. This is wrong because after fuse_dev_do_read() unlocks the FUSE request, the userspace filesystem can mark the request as completed, causing write() to return. At that point, the userspace filesystem should no longer have access to the pipe buffer. Fix by copying pages coming from the user address space to new pipe buffers. Reported-by: Jann Horn [email protected] Fixes: c3021629a0d8 (“fuse: support splice() reading from fuse device”) Cc: [email protected] Signed-off-by: Miklos Szeredi [email protected]

diff --git a/fs/fuse/dev.c b/fs/fuse/dev.c
index cd54a529460da…592730fd6e424 100644
— a/fs/fuse/dev.c
+++ b/fs/fuse/dev.c

@@ -941,7 +941,17 @@ static int fuse_copy_page(struct fuse_copy_state *cs, struct page **pagep,

while (count) {

if (cs->write && cs->pipebufs && page) {

- return fuse_ref_page(cs, page, offset, count);

+ /*

+ * Can’t control lifetime of pipe buffers, so always

+ * copy user pages.

+ */

+ if (cs->req->args->user_pages) {

+ err = fuse_copy_fill(cs);

+ if (err)

+ return err;

+ } else {

+ return fuse_ref_page(cs, page, offset, count);

+ }

} else if (!cs->len) {

if (cs->move_pages && page &&

offset == 0 && count == PAGE_SIZE) {

diff --git a/fs/fuse/file.c b/fs/fuse/file.c
index 8290944517749…0fc150c1c50bd 100644
— a/fs/fuse/file.c
+++ b/fs/fuse/file.c

@@ -1413,6 +1413,7 @@ static int fuse_get_user_pages(struct fuse_args_pages *ap, struct iov_iter *ii,

(PAGE_SIZE - ret) & (PAGE_SIZE - 1);

}

+ ap->args.user_pages = true;

if (write)

ap->args.in_pages = true;

else

diff --git a/fs/fuse/fuse_i.h b/fs/fuse/fuse_i.h
index e8e59fbdefebe…eac4984cc753c 100644
— a/fs/fuse/fuse_i.h
+++ b/fs/fuse/fuse_i.h

@@ -256,6 +256,7 @@ struct fuse_args {

bool nocreds:1;

bool in_pages:1;

bool out_pages:1;

+ bool user_pages:1;

bool out_argvar:1;

bool page_zeroing:1;

bool page_replace:1;