summaryrefslogtreecommitdiff
path: root/block/fops.c
diff options
context:
space:
mode:
authorYu Kuai <yukuai3@huawei.com>2024-09-09 16:41:48 +0300
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>2024-10-17 16:22:28 +0300
commitde0456460f2abf921e356ed2bd8da87a376680bd (patch)
tree86f759ef92956b0a8e88e691ae336269b73616f4 /block/fops.c
parent7c21e985399fc8cdcd78f18c28da6508684cace3 (diff)
downloadlinux-de0456460f2abf921e356ed2bd8da87a376680bd.tar.xz
block, bfq: fix uaf for accessing waker_bfqq after splitting
commit 1ba0403ac6447f2d63914fb760c44a3b19c44eaf upstream. After commit 42c306ed7233 ("block, bfq: don't break merge chain in bfq_split_bfqq()"), if the current procress is the last holder of bfqq, the bfqq can be freed after bfq_split_bfqq(). Hence recored the bfqq and then access bfqq->waker_bfqq may trigger UAF. What's more, the waker_bfqq may in the merge chain of bfqq, hence just recored waker_bfqq is still not safe. Fix the problem by adding a helper bfq_waker_bfqq() to check if bfqq->waker_bfqq is in the merge chain, and current procress is the only holder. Fixes: 42c306ed7233 ("block, bfq: don't break merge chain in bfq_split_bfqq()") Signed-off-by: Yu Kuai <yukuai3@huawei.com> Link: https://lore.kernel.org/r/20240909134154.954924-2-yukuai1@huaweicloud.com Signed-off-by: Jens Axboe <axboe@kernel.dk> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Diffstat (limited to 'block/fops.c')
0 files changed, 0 insertions, 0 deletions