В ядре Linux устранена следующая уязвимость:
блок: остановить таймер тайм-аута при освобождении никогда не добавленного диска
disk_release() отменяет blk_mq_init_allocated_queue() для диска,
проверка не удалась до add_disk(), но она вызывает только blk_mq_exit_queue(). Ничто не останавливает q->timeout, и этот таймер перекатывается вперед: он остается
ожидает до истечения следующего срока его действия, а не до завершения последнего запроса. Таким образом, если драйвер выполнил какие-либо операции ввода-вывода перед добавлением диска,
request_queue освобождается, хотя он все еще связан с сегментом колеса таймера.
Коммит 6f8191fdf41d («блокировать: упростить выключение диска») удалил
blk_cleanup_queue(), который раньше останавливал его. __del_gendisk() и
blk_mq_destroy_queue() все еще работает; только путь отказа зонда потерял его.
nvme попадает туда, потому что nvme_update_ns_info() отправляет зоны отчета или
FDP io-mgmt-recv в очереди ns-> перед добавлением диска, поэтому позже
сбой - одновременный сброс настроек NVME_CTRL_FROZEN или
Ошибка устройства_add_disk() — попадание в put_disk() с активированным таймером:
ОШИБКА: KASAN: slab-use-after-free в detach_if_pending+0x30c/0x340
Запись размера 8 по адресу ffff888004d71310 с помощью задачи kworker/u8:2/37.
__timer_delete_sync+0x156/0x240 ядро/время/timer.c:1621
blk_sync_queue+0x22/0x40 блок/blk-core.c:222
nvme_sync_queues+0x100/0x150 драйверы/nvme/host/core.c:5362
nvme_reset_work+0x138/0x930 драйверы/nvme/host/pci.c:3264
Выделено заданием 34:
__blk_mq_alloc_disk+0x33/0x100 блок/blk-mq.c:4462
nvme_alloc_ns+0x290/0x3870 драйверы/nvme/host/core.c:4146
Освобожден задачей 0:
blk_free_queue_rcu+0x3a/0x50 блок/blk-core.c:254
rcu_core+0xc10/0x1730 ядро/rcu/tree.c:2857
Здесь синхронизируется очередь ctrl->admin_q, только жертва поделилась
сегмент колеса таймера с висящей записью освобожденной очереди; другие пробеги
сработало в enqueue_timer(), __run_timers() или blk_mq_timeout_work(). Ошибка nvme_alloc_ns() с патчем отладки делает его детерминированным: один
утечка таймера отключает KASAN за считанные секунды, а исправленные выпуски 1987 года
не произвело никаких брызг. Остановите таймер и рабочие элементы очереди перед blk_mq_exit_queue(), например
blk_mq_destroy_queue() делает.
Найдено FuzzNvme.
Показать оригинальное описание (EN)
In the Linux kernel, the following vulnerability has been resolved: block: stop the timeout timer when releasing a never added disk disk_release() undoes blk_mq_init_allocated_queue() for a disk whose probe failed before add_disk(), but it only calls blk_mq_exit_queue(). Nothing there stops q->timeout, and that timer rolls forward: it stays pending until it next expires, not until the last request completes. So if the driver issued any I/O before adding the disk, the request_queue is freed while still linked into a timer wheel bucket. Commit 6f8191fdf41d ("block: simplify disk shutdown") dropped the blk_cleanup_queue() call that used to stop it. __del_gendisk() and blk_mq_destroy_queue() still do; only the probe failure path lost it. nvme gets there because nvme_update_ns_info() submits Report Zones or FDP io-mgmt-recv on ns->queue before the disk is added, so a later failure - a concurrent reset setting NVME_CTRL_FROZEN, or device_add_disk() failing - lands in put_disk() with the timer armed: BUG: KASAN: slab-use-after-free in detach_if_pending+0x30c/0x340 Write of size 8 at addr ffff888004d71310 by task kworker/u8:2/37 __timer_delete_sync+0x156/0x240 kernel/time/timer.c:1621 blk_sync_queue+0x22/0x40 block/blk-core.c:222 nvme_sync_queues+0x100/0x150 drivers/nvme/host/core.c:5362 nvme_reset_work+0x138/0x930 drivers/nvme/host/pci.c:3264 Allocated by task 34: __blk_mq_alloc_disk+0x33/0x100 block/blk-mq.c:4462 nvme_alloc_ns+0x290/0x3870 drivers/nvme/host/core.c:4146 Freed by task 0: blk_free_queue_rcu+0x3a/0x50 block/blk-core.c:254 rcu_core+0xc10/0x1730 kernel/rcu/tree.c:2857 The queue being synced there is ctrl->admin_q, only a victim sharing a timer wheel bucket with the freed queue's dangling entry; other runs tripped in enqueue_timer(), __run_timers() or blk_mq_timeout_work(). Failing nvme_alloc_ns() with a debug patch makes it deterministic: one leaked timer trips KASAN within seconds, while 1987 patched releases produced no splat. Stop the timer and the queue work items before blk_mq_exit_queue(), like blk_mq_destroy_queue() does. Found by FuzzNvme.