nbd: fix uaf in nbd_open
authorLi Lingfeng <lilingfeng3@huawei.com>
Tue, 7 Nov 2023 10:34:35 +0000 (18:34 +0800)
committerJens Axboe <axboe@kernel.dk>
Tue, 7 Nov 2023 15:14:57 +0000 (08:14 -0700)
commit327462725b0f759f093788dfbcb2f1fd132f956b
treead6437b4535a62a854f869c0050eea1491b2b617
parentd2f51b3516dade79269ff45eae2a7668ae711b25
nbd: fix uaf in nbd_open

Commit 4af5f2e03013 ("nbd: use blk_mq_alloc_disk and
blk_cleanup_disk") cleans up disk by blk_cleanup_disk() and it won't set
disk->private_data as NULL as before. UAF may be triggered in nbd_open()
if someone tries to open nbd device right after nbd_put() since nbd has
been free in nbd_dev_remove().

Fix this by implementing ->free_disk and free private data in it.

Fixes: 4af5f2e03013 ("nbd: use blk_mq_alloc_disk and blk_cleanup_disk")
Signed-off-by: Li Lingfeng <lilingfeng3@huawei.com>
Reviewed-by: Josef Bacik <josef@toxicpanda.com>
Link: https://lore.kernel.org/r/20231107103435.2074904-1-lilingfeng@huaweicloud.com
Signed-off-by: Jens Axboe <axboe@kernel.dk>
drivers/block/nbd.c