diff options
author | Yu Kuai <yukuai3@huawei.com> | 2023-08-08 18:49:12 +0800 |
---|---|---|
committer | Song Liu <song@kernel.org> | 2023-08-15 09:40:27 -0700 |
commit | 0d0bd28c500173bfca78aa840f8f36d261ef1765 (patch) | |
tree | aeac1e3759a484a8586396ed92f80edc76947792 /mm | |
parent | 7b3c70c43c13ad8e59f5561b154663d6bdb77021 (diff) | |
download | linux-starfive-0d0bd28c500173bfca78aa840f8f36d261ef1765.tar.gz linux-starfive-0d0bd28c500173bfca78aa840f8f36d261ef1765.tar.bz2 linux-starfive-0d0bd28c500173bfca78aa840f8f36d261ef1765.zip |
md/raid5-cache: fix null-ptr-deref for r5l_flush_stripe_to_raid()
r5l_flush_stripe_to_raid() will check if the list 'flushing_ios' is
empty, and then submit 'flush_bio', however, r5l_log_flush_endio()
is clearing the list first and then clear the bio, which will cause
null-ptr-deref:
T1: submit flush io
raid5d
handle_active_stripes
r5l_flush_stripe_to_raid
// list is empty
// add 'io_end_ios' to the list
bio_init
submit_bio
// io1
T2: io1 is done
r5l_log_flush_endio
list_splice_tail_init
// clear the list
T3: submit new flush io
...
r5l_flush_stripe_to_raid
// list is empty
// add 'io_end_ios' to the list
bio_init
bio_uninit
// clear bio->bi_blkg
submit_bio
// null-ptr-deref
Fix this problem by clearing bio before clearing the list in
r5l_log_flush_endio().
Fixes: 0dd00cba99c3 ("raid5-cache: fully initialize flush_bio when needed")
Reported-and-tested-by: Corey Hickey <bugfood-ml@fatooh.org>
Closes: https://lore.kernel.org/all/cddd7213-3dfd-4ab7-a3ac-edd54d74a626@fatooh.org/
Signed-off-by: Yu Kuai <yukuai3@huawei.com>
Reviewed-by: Christoph Hellwig <hch@lst.de>
Signed-off-by: Song Liu <song@kernel.org>
Diffstat (limited to 'mm')
0 files changed, 0 insertions, 0 deletions