В ядре Linux устранена следующая уязвимость:
USB: xhci: выйти из настройки, если контроллер недоступен
xhci_gen_setup() находит операционные регистры, используя возможность
длина считывается с самого первого регистра:
xhci->op_regs = hcd->regs +
HC_LENGTH(readl(&xhci->cap_regs->hc_capbase));
Если контроллер не работает или выпал из шины, чтение возвращается.
~0, HC_LENGTH() усекает его до 0xff, а op_regs заканчивает 0xff байтами.
мимо базы MMIO, выровненной по странице, т. е. без выравнивания. Первый доступ
через него xhci_halt() -> xhci_handshake() читает op_regs->status,
затем невыровненный readl() в памяти устройства. ошибки Arm64 при невыровнении
доступ к устройству, поэтому вместо xhci_handshake(), перехватывающего все
значение и возврат -ENODEV, установка не работает:
xhci-pci-renesas 0005:08:00.0: Невозможно изменить состояние питания с D3cold на D0, устройство недоступно
xhci-pci-renesas 0005:08:00.0: Хост-контроллер xHCI
xhci-pci-renesas 0005:08:00.0: зарегистрирована новая шина USB, присвоен номер шины 1. Невозможно обработать запрос на подкачку ядра по виртуальному адресу ffff80030a770103.
СОЭ = 0x0000000096000021
FSC = 0x21: ошибка выравнивания
Внутренняя ошибка: К сожалению: 0000000096000021 [#1] SMP
ПК: xhci_halt [xhci_hcd]
Трассировка звонков:
xhci_halt
xhci_gen_setup
xhci_pci_setup
usb_add_hcd
usb_hcd_pci_probe
xhci_pci_common_probe
xhci_pci_renesas_probe
Это произошло из-за того, что Renesas uPD720201 не включился («Unable
изменить состояние питания с D3cold на D0, устройство недоступно"), но все еще
достиг пути зонда HCD. Прочитайте регистр возможностей один раз, и если он прочитает обратно, все единицы
значение (как уже проверено xhci_handshake() и xhci_reset()), прервать
настройте с помощью -ENODEV до того, как на его основе будет получен op_regs. Прочитав один раз
также позволяет избежать повторного чтения регистра, который может измениться при одновременном выполнении
горячее удаление.
Показать оригинальное описание (EN)
In the Linux kernel, the following vulnerability has been resolved: usb: xhci: bail out of setup if the controller is inaccessible xhci_gen_setup() locates the operational registers using the capability length read from the very first register: xhci->op_regs = hcd->regs + HC_LENGTH(readl(&xhci->cap_regs->hc_capbase)); If the controller is dead or has dropped off the bus, that read returns ~0, HC_LENGTH() truncates it to 0xff, and op_regs ends up 0xff bytes past the page-aligned MMIO base, i.e. unaligned. The first access through it, xhci_halt() -> xhci_handshake() reading op_regs->status, is then an unaligned readl() on device memory. arm64 faults on unaligned device accesses, so instead of xhci_handshake() catching the all-ones value and returning -ENODEV, setup oopses: xhci-pci-renesas 0005:08:00.0: Unable to change power state from D3cold to D0, device inaccessible xhci-pci-renesas 0005:08:00.0: xHCI Host Controller xhci-pci-renesas 0005:08:00.0: new USB bus registered, assigned bus number 1 Unable to handle kernel paging request at virtual address ffff80030a770103 ESR = 0x0000000096000021 FSC = 0x21: alignment fault Internal error: Oops: 0000000096000021 [#1] SMP pc : xhci_halt [xhci_hcd] Call trace: xhci_halt xhci_gen_setup xhci_pci_setup usb_add_hcd usb_hcd_pci_probe xhci_pci_common_probe xhci_pci_renesas_probe This was hit with a Renesas uPD720201 that failed to power up ("Unable to change power state from D3cold to D0, device inaccessible") yet still reached the HCD probe path. Read the capability register once, and if it reads back the all-ones value (as xhci_handshake() and xhci_reset() already test for), abort setup with -ENODEV before op_regs is derived from it. Reading it once also avoids re-reading a register that may change under a concurrent hot-removal.