В ядре Linux устранена следующая уязвимость:
wifi: libertas: исправлено использование после освобождения в lbs_free_adapter()
Функция lbs_free_adapter() использует timer_delete() (несинхронно).
как для command_timer, так и для tx_lockup_timer до того, как структура будет
освобожден. Это неверно, поскольку timer_delete() не ждет
любой работающий обратный вызов таймера для завершения. Если обратный вызов таймера выполняется при вызове lbs_free_adapter(),
обратный вызов получит доступ к освобожденной памяти, поскольку lbs_cfg_free() освобождает
содержащий структуру сразу после возврата lbs_free_adapter().
Оба обратных вызова таймера (lbs_cmd_timeout_handler и lbs_tx_lockup_handler)
получить доступ к priv->driver_lock, priv->cur_cmd, priv->dev и другим полям,
все это будет нарушением использования после освобождения. Вместо этого используйте timer_delete_sync(), чтобы гарантировать выполнение любого обратного вызова таймера.
завершился перед возвращением. Эта ошибка была обнаружена в коммите 8f641d93c38a («libertas: обнаружить TX
блокировки и сброс оборудования"), где вместо del_timer() использовалась функция
del_timer_sync() в пути очистки.
Command_timer имеет та же проблема, так как драйвер был впервые написан.
Показать оригинальное описание (EN)
In the Linux kernel, the following vulnerability has been resolved: wifi: libertas: fix use-after-free in lbs_free_adapter() The lbs_free_adapter() function uses timer_delete() (non-synchronous) for both command_timer and tx_lockup_timer before the structure is freed. This is incorrect because timer_delete() does not wait for any running timer callback to complete. If a timer callback is executing when lbs_free_adapter() is called, the callback will access freed memory since lbs_cfg_free() frees the containing structure immediately after lbs_free_adapter() returns. Both timer callbacks (lbs_cmd_timeout_handler and lbs_tx_lockup_handler) access priv->driver_lock, priv->cur_cmd, priv->dev, and other fields, which would all be use-after-free violations. Use timer_delete_sync() instead to ensure any running timer callback has completed before returning. This bug was introduced in commit 8f641d93c38a ("libertas: detect TX lockups and reset hardware") where del_timer() was used instead of del_timer_sync() in the cleanup path. The command_timer has had the same issue since the driver was first written.