Skip to content

Commit cac3419

Browse files
author
CKI Backport Bot
committed
wifi: ath12k: fix uaf in ath12k_core_init()
JIRA: https://issues.redhat.com/browse/RHEL-102086 CVE: CVE-2025-38116 commit f3fe49d Author: Miaoqing Pan <miaoqing.pan@oss.qualcomm.com> Date: Wed Jun 4 13:52:50 2025 +0800 wifi: ath12k: fix uaf in ath12k_core_init() When the execution of ath12k_core_hw_group_assign() or ath12k_core_hw_group_create() fails, the registered notifier chain is not unregistered properly. Its memory is freed after rmmod, which may trigger to a use-after-free (UAF) issue if there is a subsequent access to this notifier chain. Fixes the issue by calling ath12k_core_panic_notifier_unregister() in failure cases. Call trace: notifier_chain_register+0x4c/0x1f0 (P) atomic_notifier_chain_register+0x38/0x68 ath12k_core_init+0x50/0x4e8 [ath12k] ath12k_pci_probe+0x5f8/0xc28 [ath12k] pci_device_probe+0xbc/0x1a8 really_probe+0xc8/0x3a0 __driver_probe_device+0x84/0x1b0 driver_probe_device+0x44/0x130 __driver_attach+0xcc/0x208 bus_for_each_dev+0x84/0x100 driver_attach+0x2c/0x40 bus_add_driver+0x130/0x260 driver_register+0x70/0x138 __pci_register_driver+0x68/0x80 ath12k_pci_init+0x30/0x68 [ath12k] ath12k_init+0x28/0x78 [ath12k] Tested-on: WCN7850 hw2.0 PCI WLAN.HMT.1.0.c5-00481-QCAHMTSWPL_V1.0_V2.0_SILICONZ-3 Fixes: 6f245ea ("wifi: ath12k: introduce device group abstraction") Signed-off-by: Miaoqing Pan <miaoqing.pan@oss.qualcomm.com> Reviewed-by: Baochen Qiang <quic_bqiang@quicinc.com> Link: https://patch.msgid.link/20250604055250.1228501-1-miaoqing.pan@oss.qualcomm.com Signed-off-by: Jeff Johnson <jeff.johnson@oss.qualcomm.com> Signed-off-by: CKI Backport Bot <cki-ci-bot+cki-gitlab-backport-bot@redhat.com>
1 parent 134517e commit cac3419

File tree

1 file changed

+7
-3
lines changed
  • drivers/net/wireless/ath/ath12k

1 file changed

+7
-3
lines changed

drivers/net/wireless/ath/ath12k/core.c

Lines changed: 7 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1843,7 +1843,8 @@ int ath12k_core_init(struct ath12k_base *ab)
18431843
if (!ag) {
18441844
mutex_unlock(&ath12k_hw_group_mutex);
18451845
ath12k_warn(ab, "unable to get hw group\n");
1846-
return -ENODEV;
1846+
ret = -ENODEV;
1847+
goto err_unregister_notifier;
18471848
}
18481849

18491850
mutex_unlock(&ath12k_hw_group_mutex);
@@ -1858,17 +1859,20 @@ int ath12k_core_init(struct ath12k_base *ab)
18581859
if (ret) {
18591860
mutex_unlock(&ag->mutex);
18601861
ath12k_warn(ab, "unable to create hw group\n");
1861-
goto err;
1862+
goto err_destroy_hw_group;
18621863
}
18631864
}
18641865

18651866
mutex_unlock(&ag->mutex);
18661867

18671868
return 0;
18681869

1869-
err:
1870+
err_destroy_hw_group:
18701871
ath12k_core_hw_group_destroy(ab->ag);
18711872
ath12k_core_hw_group_unassign(ab);
1873+
err_unregister_notifier:
1874+
ath12k_core_panic_notifier_unregister(ab);
1875+
18721876
return ret;
18731877
}
18741878

0 commit comments

Comments
 (0)