Commit 53cd9ae8 authored by Jens Axboe's avatar Jens Axboe

[PATCH] splice: fix shadow[] filling logic

Clear the entire range, and don't increment pidx or we keep filling
the same position again and again.

Thanks to KAMEZAWA Hiroyuki.
Signed-off-by: default avatarJens Axboe <axboe@suse.de>
parent b043b673
...@@ -237,9 +237,9 @@ static int __generic_file_splice_read(struct file *in, struct inode *pipe, ...@@ -237,9 +237,9 @@ static int __generic_file_splice_read(struct file *in, struct inode *pipe,
* fill shadow[] with pages at the right locations, so we only * fill shadow[] with pages at the right locations, so we only
* have to fill holes * have to fill holes
*/ */
memset(shadow, 0, i * sizeof(struct page *)); memset(shadow, 0, nr_pages * sizeof(struct page *));
for (j = 0, pidx = index; j < i; pidx++, j++) for (j = 0; j < i; j++)
shadow[pages[j]->index - pidx] = pages[j]; shadow[pages[j]->index - index] = pages[j];
/* /*
* now fill in the holes * now fill in the holes
......
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