md: bcache: check the return value of kzalloc() in detached_dev_do_request()
commit 40f567bbb3b0639d2ec7d1c6ad4b1b018f80cf19 upstream.
The function kzalloc() in detached_dev_do_request() can fail, so its
return value should be checked.
Fixes: bc082a55d2
("bcache: fix inaccurate io state for detached bcache devices")
Reported-by: TOTE Robot <oslab@tsinghua.edu.cn>
Signed-off-by: Jia-Ju Bai <baijiaju1990@gmail.com>
Signed-off-by: Coly Li <colyli@suse.de>
Link: https://lore.kernel.org/r/20220527152818.27545-4-colyli@suse.de
Signed-off-by: Jens Axboe <axboe@kernel.dk>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
This commit is contained in:

committed by
Greg Kroah-Hartman

parent
a67100f426
commit
47c1680e51
@@ -1109,6 +1109,12 @@ static void detached_dev_do_request(struct bcache_device *d, struct bio *bio)
|
|||||||
* which would call closure_get(&dc->disk.cl)
|
* which would call closure_get(&dc->disk.cl)
|
||||||
*/
|
*/
|
||||||
ddip = kzalloc(sizeof(struct detached_dev_io_private), GFP_NOIO);
|
ddip = kzalloc(sizeof(struct detached_dev_io_private), GFP_NOIO);
|
||||||
|
if (!ddip) {
|
||||||
|
bio->bi_status = BLK_STS_RESOURCE;
|
||||||
|
bio->bi_end_io(bio);
|
||||||
|
return;
|
||||||
|
}
|
||||||
|
|
||||||
ddip->d = d;
|
ddip->d = d;
|
||||||
/* Count on the bcache device */
|
/* Count on the bcache device */
|
||||||
ddip->start_time = part_start_io_acct(d->disk, &ddip->part, bio);
|
ddip->start_time = part_start_io_acct(d->disk, &ddip->part, bio);
|
||||||
|
Reference in New Issue
Block a user