diff options
author | FUJITA Tomonori <fujita.tomonori@lab.ntt.co.jp> | 2009-04-28 14:24:29 -0400 |
---|---|---|
committer | Jens Axboe <jens.axboe@oracle.com> | 2009-04-28 14:24:29 -0400 |
commit | 69838727bcd819a8fd73a88447801221788b0c6d (patch) | |
tree | 4fd6c31c1078ac079ea1ba0fa9cdf4aa581f06a7 /fs | |
parent | 0191944282e84931f92915b5f06b348a92dac7e1 (diff) |
bio: fix memcpy corruption in bio_copy_user_iov()
st driver uses blk_rq_map_user() in order to just build a request out
of page frames. In this case, map_data->offset is a non zero value and
iov[0].iov_base is NULL. We need to increase nr_pages for that.
Cc: stable@kernel.org
Signed-off-by: FUJITA Tomonori <fujita.tomonori@lab.ntt.co.jp>
Signed-off-by: Jens Axboe <jens.axboe@oracle.com>
Diffstat (limited to 'fs')
-rw-r--r-- | fs/bio.c | 3 |
1 files changed, 3 insertions, 0 deletions
@@ -817,6 +817,9 @@ struct bio *bio_copy_user_iov(struct request_queue *q, | |||
817 | len += iov[i].iov_len; | 817 | len += iov[i].iov_len; |
818 | } | 818 | } |
819 | 819 | ||
820 | if (offset) | ||
821 | nr_pages++; | ||
822 | |||
820 | bmd = bio_alloc_map_data(nr_pages, iov_count, gfp_mask); | 823 | bmd = bio_alloc_map_data(nr_pages, iov_count, gfp_mask); |
821 | if (!bmd) | 824 | if (!bmd) |
822 | return ERR_PTR(-ENOMEM); | 825 | return ERR_PTR(-ENOMEM); |