Skip to content

Commit 8577ded

Browse files
Miklos Szeredigregkh
authored andcommitted
fuse: prevent overflow in copy_file_range return value
commit 1e08938 upstream. The FUSE protocol uses struct fuse_write_out to convey the return value of copy_file_range, which is restricted to uint32_t. But the COPY_FILE_RANGE interface supports a 64-bit size copies. Currently the number of bytes copied is silently truncated to 32-bit, which may result in poor performance or even failure to copy in case of truncation to zero. Reported-by: Florian Weimer <fweimer@redhat.com> Closes: https://lore.kernel.org/all/lhuh5ynl8z5.fsf@oldenburg.str.redhat.com/ Fixes: 88bc7d5 ("fuse: add support for copy_file_range()") Cc: <stable@vger.kernel.org> # v4.20 Signed-off-by: Miklos Szeredi <mszeredi@redhat.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
1 parent 01e1eba commit 8577ded

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed

fs/fuse/file.c

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -3106,7 +3106,7 @@ static ssize_t __fuse_copy_file_range(struct file *file_in, loff_t pos_in,
31063106
.nodeid_out = ff_out->nodeid,
31073107
.fh_out = ff_out->fh,
31083108
.off_out = pos_out,
3109-
.len = len,
3109+
.len = min_t(size_t, len, UINT_MAX & PAGE_MASK),
31103110
.flags = flags
31113111
};
31123112
struct fuse_write_out outarg;

0 commit comments

Comments
 (0)