В ядре Linux устранена следующая уязвимость:
можно: isotp: исправить гонку use-after-free с одновременным NETDEV_UNREGISTER
isotp_release() просмотрел привязанное сетевое устройство через dev_get_by_index()
используя сохраненный ifindex. Во время отмены регистрации устройство
исключен из хеша ifindex перед уведомителем NETDEV_UNREGISTER
цепочка работает, поэтому одновременная функция isotp_release() не может найти устройство, пропустите
can_rx_unregister() полностью и продолжить освобождение сокета. Поскольку isotp_release() уже удалил себя из isotp
список уведомлений в этот момент isotp_notify() никогда не получит возможности
очистите либо, оставив устаревший CAN-фильтр, который продолжает указывать на
освободившаяся розетка.
Исправьте это так же, как это уже делает raw.c: сохраните отслеживаемую ссылку на
связанное net_device в сокете (so->dev/so->dev_tracker) из
bind() и далее вместо повторного разрешения его из ifindex, и
сериализовать связывание()/release() с помощью rtnl_lock(), чтобы so->dev всегда было
соответствует тому, что видит уведомитель NETDEV_UNREGISTER. так->разработчик
остается действительным независимо от удаления ifindex-хеша из списка и действует только когда-либо.
очищается в зависимости от того, какой из isotp_release()/isotp_notify() туда попадает
во-первых, поэтому фильтр всегда снимается ровно один раз.
isotp_bind() теперь отклоняет (повторную) привязку с помощью -EAGAIN, пока so->[tx|rx].state
еще не ISOTP_IDLE, поэтому таймер, оставленный включенным ранее
NETDEV_UNREGISTER не может действовать с вновь привязанным so->ifindex. Оба чека
используют один и тот же раздел lock_sock(), поэтому нет окна, в котором
одновременная очистка isotp_notify() so->bound может быть пропущена.
Показать оригинальное описание (EN)
In the Linux kernel, the following vulnerability has been resolved: can: isotp: fix use-after-free race with concurrent NETDEV_UNREGISTER isotp_release() looked up the bound network device via dev_get_by_index() using the stored ifindex. During device unregistration the device is unlisted from the ifindex hash before the NETDEV_UNREGISTER notifier chain runs, so a concurrent isotp_release() could find no device, skip can_rx_unregister() entirely, and still proceed to free the socket. Since isotp_release() had already removed itself from the isotp notifier list at that point, isotp_notify() would never get a chance to clean up either, leaving a stale CAN filter that keeps pointing at the freed socket. Fix this the same way raw.c already does: hold a tracked reference to the bound net_device in the socket (so->dev/so->dev_tracker) from bind() onward instead of re-resolving it from the ifindex, and serialize bind()/release() with rtnl_lock() so that so->dev is always consistent with what the NETDEV_UNREGISTER notifier sees. so->dev stays valid regardless of ifindex-hash unlisting, and is only ever cleared by whichever of isotp_release()/isotp_notify() gets there first, so the filter is always removed exactly once. isotp_bind() now rejects a (re)bind with -EAGAIN while so->[tx|rx].state isn't ISOTP_IDLE yet, so a timer left running by a prior NETDEV_UNREGISTER can't act on a newly bound so->ifindex. Both checks share the same lock_sock() section, so there is no window in which a concurrent isotp_notify() clearing so->bound could be missed.
Характеристики атаки
Последствия
Строка CVSS v3.1