В ядре Linux устранена следующая уязвимость:
ipmi:si: исправлено разыменование нулевого указателя после неудачной регистрации.
try_smi_init() выделяет new_smi->si_sm и последующие вызовы
ipmi_register_smi_mod(), который соответствует ipmi_add_smi(). Во время ipmi_add_smi() верхний обработчик сообщений IPMI получает
исходную информацию об устройстве BMC через __bmc_get_device_id(). Это может
сбой, если BMC не возвращает успешный ответ на запрос «Получить устройство».
ID-команда. Когда BMC возвращает ненулевой код завершения, помощник по идентификатору устройства
повторяет команду и в конечном итоге возвращает -EIO, если идентификатор устройства все еще
невозможно забрать. На этом пути сбоя ipmi_add_smi() регистрирует «Невозможно получить идентификатор устройства».
и переходит к out_err_started, где вызывает драйвер нижнего уровня.
обратный вызов для завершения работы. try_smi_init() затем регистрирует возвращенную регистрацию.
неудача:
ipmi_si IPI0001:00: Обработчик сообщений IPMI: невозможно получить идентификатор устройства: -5
ipmi_si IPI0001:00: Невозможно зарегистрировать устройство: ошибка -5
Для ipmi_si обратным вызовом завершения работы является Shutdown_smi(), который очищает
данные конечного автомата SI, освобождает smi_info->si_sm и устанавливает
smi_info->si_sm и smi_info->intf в NULL.
Однако intf->in_shutdown не установлен при этой неудачной регистрации.
путь отката. Таким образом, асинхронный рабочий элемент redo_bmc_reg может
все равно повторить попытку проверки идентификатора устройства BMC после того, как нижний драйвер уже выполнил
очистил данные своего конечного автомата SI. В наблюдаемом случае этот путь повторной попытки
достиг start_next_msg(), который передал NULL указатель smi_info->si_sm
к выбранному обработчику конечного автомата KCS:
ОШИБКА: невозможно обработать разыменование нулевого указателя ядра по адресу 0000000000000000.
Рабочая очередь: события redo_bmc_reg [ipmi_msghandler]
RIP: start_kcs_transaction+0x2c/0x190 [ipmi_si]
Отслеживание вызова:
start_next_msg+0x50/0x80 [ipmi_si]
check_start_timer_thread.part.9+0x3b/0x50 [ipmi_si]
отправитель+0x69/0x80 [ipmi_si]
i_ipmi_request+0x2ac/0x9d0 [ipmi_msghandler]
__get_device_id.isra.29+0xaa/0x180 [ipmi_msghandler]
__bmc_get_device_id+0xef/0x950 [ipmi_msghandler]
redo_bmc_reg+0x52/0x60 [ipmi_msghandler]
процесс_one_work+0x1a7/0x360
Установите intf->in_shutdown по пути out_err_started перед вызовом
обратный вызов при выключении нижнего драйвера. Это предотвращает последующее выполнение redo_bmc_reg.
пытается использовать интерфейс, нижнее состояние драйвера которого было
очищен и применяет то же состояние завершения работы к другим интерфейсам IPMI.
также.
Показать оригинальное описание (EN)
In the Linux kernel, the following vulnerability has been resolved: ipmi: si: Fix NULL pointer dereference after failed registration try_smi_init() allocates new_smi->si_sm and later calls ipmi_register_smi_mod(), which maps to ipmi_add_smi(). During ipmi_add_smi(), the upper IPMI message handler obtains the initial BMC device information through __bmc_get_device_id(). This can fail if the BMC does not return a successful response to the Get Device ID command. When the BMC returns a nonzero completion code, the device-id helper retries the command and eventually returns -EIO if the device ID still cannot be fetched. On this failure path, ipmi_add_smi() logs "Unable to get the device id" and goes to out_err_started, where it invokes the lower driver's shutdown callback. try_smi_init() then logs the returned registration failure: ipmi_si IPI0001:00: IPMI message handler: Unable to get the device id: -5 ipmi_si IPI0001:00: Unable to register device: error -5 For ipmi_si, the shutdown callback is shutdown_smi(), which cleans up the SI state machine data, frees smi_info->si_sm, and sets smi_info->si_sm and smi_info->intf to NULL. However, intf->in_shutdown is not set on this failed-registration rollback path. Therefore, the asynchronous redo_bmc_reg work item can still retry BMC device-id probing after the lower driver has already cleared its SI state machine data. In the observed case, that retry path reached start_next_msg(), which passed the NULL smi_info->si_sm pointer to the selected KCS state machine handler: BUG: unable to handle kernel NULL pointer dereference at 0000000000000000 Workqueue: events redo_bmc_reg [ipmi_msghandler] RIP: start_kcs_transaction+0x2c/0x190 [ipmi_si] Call Trace: start_next_msg+0x50/0x80 [ipmi_si] check_start_timer_thread.part.9+0x3b/0x50 [ipmi_si] sender+0x69/0x80 [ipmi_si] i_ipmi_request+0x2ac/0x9d0 [ipmi_msghandler] __get_device_id.isra.29+0xaa/0x180 [ipmi_msghandler] __bmc_get_device_id+0xef/0x950 [ipmi_msghandler] redo_bmc_reg+0x52/0x60 [ipmi_msghandler] process_one_work+0x1a7/0x360 Set intf->in_shutdown on the out_err_started path before invoking the lower driver's shutdown callback. This prevents later redo_bmc_reg retries from using an interface whose lower driver state has been cleaned up, and applies the same shutdown state to other IPMI interfaces as well.