В ядре Linux устранена следующая уязвимость:
net/mlx5e: исправлена гонка публикаций для priv->channel_stats[]
mlx5e_channel_stats_alloc() публикует новую запись в
priv->channel_stats[], а затем увеличивает priv->stats_nch как
токен публикации, но ни одно из хранилищ не имеет барьера памяти:
priv->channel_stats[ix] = kvzalloc_node(...);
if (!priv->channel_stats[ix])
возврат -ЭНОМЕМ;
priv->stats_nch++;
Параллельные считыватели вычисляют цикл, связанный с priv->stats_nch и
затем разыменуйте priv->channel_stats[i], используя простой доступ, например.
for (i = 0; i <priv->stats_nch; i++) {
struct mlx5e_channel_stats *cs = priv->channel_stats[i];
... cs->rq.packets ...
}
На слабоупорядоченных архитектурах (ARM, PowerPC, RISC-V) запись
Channel_stats[ix] и stats_nch могут стать видимыми для других процессоров.
порядка программы. Читатель может наблюдать stats_nch == N, пока еще
видим канал_stats[N-1] == NULL, что приводит к указателю NULL
разыменование в цикле Channel_stats. Это наблюдалось в производстве на DPU BlueField-3 (arm64),
где ovs-vswitchd запрашивает статистику netdev через netlink во время NIC
воспитание, гонки mlx5e_open_channel() -> mlx5e_channel_stats_alloc()
на другом процессоре:
Невозможно обработать разыменование нулевого указателя ядра по виртуальному адресу 0x840.
Название оборудования: BlueField-3 DPU
ПК: mlx5e_fold_sw_stats64+0x30/0x180 [mlx5_core]
Трассировка звонков:
mlx5e_fold_sw_stats64+0x30/0x180 [mlx5_core]
dev_get_stats+0x50/0xc0
ovs_vport_get_stats+0x38/0xac [openvswitch]
ovs_vport_cmd_fill_info+0x194/0x290 [openvswitch]
ovs_vport_cmd_get+0xbc/0x10c [openvswitch]
genl_family_rcv_msg_doit+0xd0/0x160
genl_rcv_msg+0xec/0x1f0
netlink_rcv_skb+0x64/0x130
genl_rcv+0x40/0x60
netlink_unicast+0x2fc/0x370
netlink_sendmsg+0x1dc/0x454
...
__arm64_sys_sendmsg+0x2c/0x40
Добавьте помощники mlx5e_stats_nch_write() и mlx5e_stats_nch_read() в en.h.
которые оборачивают пару smp_store_release()/smp_load_acquire() в stats_nch. Пара выпуска/приобретения устанавливает контракт:
stats_nch == N => Channel_stats[0..N-1] видимы и не равны NULL. Опубликуйте приращение stats_nch с помощью mlx5e_stats_nch_write() в
Writer (mlx5e_channel_stats_alloc()) и прочитайте stats_nch через
mlx5e_stats_nch_read() во всех читателях: статистика очереди mlx5e RX/TX,
mlx5e_get_base_stats(), статистика каналов ethtool, статистика IPoIB,
sw_stats и агент статистики HV VHCA.
Показать оригинальное описание (EN)
In the Linux kernel, the following vulnerability has been resolved: net/mlx5e: Fix publication race for priv->channel_stats[] mlx5e_channel_stats_alloc() publishes a new entry to priv->channel_stats[] and then increments priv->stats_nch as a publication token, but neither store carries any memory barrier: priv->channel_stats[ix] = kvzalloc_node(...); if (!priv->channel_stats[ix]) return -ENOMEM; priv->stats_nch++; Concurrent readers compute the loop bound from priv->stats_nch and then dereference priv->channel_stats[i] using plain accesses, e.g. for (i = 0; i < priv->stats_nch; i++) { struct mlx5e_channel_stats *cs = priv->channel_stats[i]; ... cs->rq.packets ... } On weakly-ordered architectures (ARM, PowerPC, RISC-V) the writes to channel_stats[ix] and stats_nch may become visible to other CPUs out of program order. A reader can observe stats_nch == N while still seeing channel_stats[N-1] == NULL, leading to a NULL pointer dereference in the channel_stats loop. This has been observed in production on BlueField-3 DPUs (arm64), where ovs-vswitchd queries netdev statistics over netlink during NIC bringup, racing mlx5e_open_channel() -> mlx5e_channel_stats_alloc() on another CPU: Unable to handle kernel NULL pointer dereference at virtual address 0x840 Hardware name: BlueField-3 DPU pc : mlx5e_fold_sw_stats64+0x30/0x180 [mlx5_core] Call trace: mlx5e_fold_sw_stats64+0x30/0x180 [mlx5_core] dev_get_stats+0x50/0xc0 ovs_vport_get_stats+0x38/0xac [openvswitch] ovs_vport_cmd_fill_info+0x194/0x290 [openvswitch] ovs_vport_cmd_get+0xbc/0x10c [openvswitch] genl_family_rcv_msg_doit+0xd0/0x160 genl_rcv_msg+0xec/0x1f0 netlink_rcv_skb+0x64/0x130 genl_rcv+0x40/0x60 netlink_unicast+0x2fc/0x370 netlink_sendmsg+0x1dc/0x454 ... __arm64_sys_sendmsg+0x2c/0x40 Add mlx5e_stats_nch_write() and mlx5e_stats_nch_read() helpers in en.h that wrap the smp_store_release()/smp_load_acquire() pair on stats_nch. The release/acquire pair establishes the contract: stats_nch == N => channel_stats[0..N-1] are visible and non-NULL. Publish the stats_nch increment via mlx5e_stats_nch_write() in the writer (mlx5e_channel_stats_alloc()), and read stats_nch via mlx5e_stats_nch_read() in all readers: mlx5e RX/TX queue stats, mlx5e_get_base_stats(), ethtool channels stats, IPoIB stats, the sw_stats fold and the HV VHCA stats agent.