summaryrefslogtreecommitdiffstats
path: root/fs
diff options
context:
space:
mode:
authorJens Axboe <axboe@suse.de>2006-04-30 16:36:32 +0200
committerJens Axboe <axboe@suse.de>2006-04-30 16:36:32 +0200
commit46e678c96bbd775abd05d3ddbe2fd334794f9157 (patch)
treef82b77d9f2fd5d4eb5e66bc2bb8ff5df63db3c7f /fs
parente0a515bc6a2188f02916e976f419a8640312e32a (diff)
downloadop-kernel-dev-46e678c96bbd775abd05d3ddbe2fd334794f9157.zip
op-kernel-dev-46e678c96bbd775abd05d3ddbe2fd334794f9157.tar.gz
[PATCH] splice: fix bugs with stealing regular pipe pages
- Check that page has suitable count for stealing in the regular pipes. - pipe_to_file() assumes that the page is locked on succesful steal, so do that in the pipe steal hook - Missing unlock_page() in add_to_page_cache() failure. Signed-off-by: Jens Axboe <axboe@suse.de>
Diffstat (limited to 'fs')
-rw-r--r--fs/pipe.c11
-rw-r--r--fs/splice.c4
2 files changed, 12 insertions, 3 deletions
diff --git a/fs/pipe.c b/fs/pipe.c
index 7fefb10..5a36927 100644
--- a/fs/pipe.c
+++ b/fs/pipe.c
@@ -127,8 +127,15 @@ static void anon_pipe_buf_unmap(struct pipe_inode_info *pipe,
static int anon_pipe_buf_steal(struct pipe_inode_info *pipe,
struct pipe_buffer *buf)
{
- buf->flags |= PIPE_BUF_FLAG_STOLEN;
- return 0;
+ struct page *page = buf->page;
+
+ if (page_count(page) == 1) {
+ buf->flags |= PIPE_BUF_FLAG_STOLEN;
+ lock_page(page);
+ return 0;
+ }
+
+ return 1;
}
static void anon_pipe_buf_get(struct pipe_inode_info *info,
diff --git a/fs/splice.c b/fs/splice.c
index a46ddd2..9df28d3 100644
--- a/fs/splice.c
+++ b/fs/splice.c
@@ -599,8 +599,10 @@ static int pipe_to_file(struct pipe_inode_info *info, struct pipe_buffer *buf,
goto find_page;
page = buf->page;
- if (add_to_page_cache(page, mapping, index, gfp_mask))
+ if (add_to_page_cache(page, mapping, index, gfp_mask)) {
+ unlock_page(page);
goto find_page;
+ }
if (!(buf->flags & PIPE_BUF_FLAG_LRU))
lru_cache_add(page);
OpenPOWER on IntegriCloud