diff options
-rw-r--r-- | queue-6.12/series | 1 | ||||
-rw-r--r-- | queue-6.12/wifi-cfg80211-init-wiphy_work-before-allocating-rfkill-fails.patch | 98 |
2 files changed, 99 insertions, 0 deletions
diff --git a/queue-6.12/series b/queue-6.12/series index 4538049c994..b1e15d08faf 100644 --- a/queue-6.12/series +++ b/queue-6.12/series @@ -328,3 +328,4 @@ x86-pkeys-simplify-pkru-update-in-signal-frame.patch cpufreq-amd-pstate-add-missing-null-ptr-check-in-amd_pstate_update.patch input-sparcspkr-avoid-unannotated-fall-through.patch wifi-ath12k-clear-affinity-hint-before-calling-ath12k_pci_free_irq-in-error-path.patch +wifi-cfg80211-init-wiphy_work-before-allocating-rfkill-fails.patch diff --git a/queue-6.12/wifi-cfg80211-init-wiphy_work-before-allocating-rfkill-fails.patch b/queue-6.12/wifi-cfg80211-init-wiphy_work-before-allocating-rfkill-fails.patch new file mode 100644 index 00000000000..a16bdc68a36 --- /dev/null +++ b/queue-6.12/wifi-cfg80211-init-wiphy_work-before-allocating-rfkill-fails.patch @@ -0,0 +1,98 @@ +From fc88dee89d7b63eeb17699393eb659aadf9d9b7c Mon Sep 17 00:00:00 2001 +From: Edward Adam Davis <eadavis@qq.com> +Date: Tue, 18 Mar 2025 11:13:45 +0800 +Subject: wifi: cfg80211: init wiphy_work before allocating rfkill fails + +From: Edward Adam Davis <eadavis@qq.com> + +commit fc88dee89d7b63eeb17699393eb659aadf9d9b7c upstream. + +syzbort reported a uninitialize wiphy_work_lock in cfg80211_dev_free. [1] + +After rfkill allocation fails, the wiphy release process will be performed, +which will cause cfg80211_dev_free to access the uninitialized wiphy_work +related data. + +Move the initialization of wiphy_work to before rfkill initialization to +avoid this issue. + +[1] +INFO: trying to register non-static key. +The code is fine but needs lockdep annotation, or maybe +you didn't initialize this object before use? +turning off the locking correctness validator. +CPU: 0 UID: 0 PID: 5935 Comm: syz-executor550 Not tainted 6.14.0-rc6-syzkaller-00103-g4003c9e78778 #0 +Hardware name: QEMU Standard PC (Q35 + ICH9, 2009), BIOS 1.16.3-debian-1.16.3-2~bpo12+1 04/01/2014 +Call Trace: + <TASK> + __dump_stack lib/dump_stack.c:94 [inline] + dump_stack_lvl+0x116/0x1f0 lib/dump_stack.c:120 + assign_lock_key kernel/locking/lockdep.c:983 [inline] + register_lock_class+0xc39/0x1240 kernel/locking/lockdep.c:1297 + __lock_acquire+0x135/0x3c40 kernel/locking/lockdep.c:5103 + lock_acquire.part.0+0x11b/0x380 kernel/locking/lockdep.c:5851 + __raw_spin_lock_irqsave include/linux/spinlock_api_smp.h:110 [inline] + _raw_spin_lock_irqsave+0x3a/0x60 kernel/locking/spinlock.c:162 + cfg80211_dev_free+0x30/0x3d0 net/wireless/core.c:1196 + device_release+0xa1/0x240 drivers/base/core.c:2568 + kobject_cleanup lib/kobject.c:689 [inline] + kobject_release lib/kobject.c:720 [inline] + kref_put include/linux/kref.h:65 [inline] + kobject_put+0x1e4/0x5a0 lib/kobject.c:737 + put_device+0x1f/0x30 drivers/base/core.c:3774 + wiphy_free net/wireless/core.c:1224 [inline] + wiphy_new_nm+0x1c1f/0x2160 net/wireless/core.c:562 + ieee80211_alloc_hw_nm+0x1b7a/0x2260 net/mac80211/main.c:835 + mac80211_hwsim_new_radio+0x1d6/0x54e0 drivers/net/wireless/virtual/mac80211_hwsim.c:5185 + hwsim_new_radio_nl+0xb42/0x12b0 drivers/net/wireless/virtual/mac80211_hwsim.c:6242 + genl_family_rcv_msg_doit+0x202/0x2f0 net/netlink/genetlink.c:1115 + genl_family_rcv_msg net/netlink/genetlink.c:1195 [inline] + genl_rcv_msg+0x565/0x800 net/netlink/genetlink.c:1210 + netlink_rcv_skb+0x16b/0x440 net/netlink/af_netlink.c:2533 + genl_rcv+0x28/0x40 net/netlink/genetlink.c:1219 + netlink_unicast_kernel net/netlink/af_netlink.c:1312 [inline] + netlink_unicast+0x53c/0x7f0 net/netlink/af_netlink.c:1338 + netlink_sendmsg+0x8b8/0xd70 net/netlink/af_netlink.c:1882 + sock_sendmsg_nosec net/socket.c:718 [inline] + __sock_sendmsg net/socket.c:733 [inline] + ____sys_sendmsg+0xaaf/0xc90 net/socket.c:2573 + ___sys_sendmsg+0x135/0x1e0 net/socket.c:2627 + __sys_sendmsg+0x16e/0x220 net/socket.c:2659 + do_syscall_x64 arch/x86/entry/common.c:52 [inline] + do_syscall_64+0xcd/0x250 arch/x86/entry/common.c:83 + +Fixes: 72d520476a2f ("wifi: cfg80211: cancel wiphy_work before freeing wiphy") +Reported-by: syzbot+aaf0488c83d1d5f4f029@syzkaller.appspotmail.com +Close: https://syzkaller.appspot.com/bug?extid=aaf0488c83d1d5f4f029 +Tested-by: syzbot+aaf0488c83d1d5f4f029@syzkaller.appspotmail.com +Signed-off-by: Edward Adam Davis <eadavis@qq.com> +Link: https://patch.msgid.link/tencent_258DD9121DDDB9DD9A1939CFAA0D8625B107@qq.com +Signed-off-by: Johannes Berg <johannes.berg@intel.com> +Signed-off-by: WangYuli <wangyuli@uniontech.com> +Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org> +--- + net/wireless/core.c | 6 +++--- + 1 file changed, 3 insertions(+), 3 deletions(-) + +--- a/net/wireless/core.c ++++ b/net/wireless/core.c +@@ -553,6 +553,9 @@ use_default_name: + INIT_WORK(&rdev->mgmt_registrations_update_wk, + cfg80211_mgmt_registrations_update_wk); + spin_lock_init(&rdev->mgmt_registrations_lock); ++ INIT_WORK(&rdev->wiphy_work, cfg80211_wiphy_work); ++ INIT_LIST_HEAD(&rdev->wiphy_work_list); ++ spin_lock_init(&rdev->wiphy_work_lock); + + #ifdef CONFIG_CFG80211_DEFAULT_PS + rdev->wiphy.flags |= WIPHY_FLAG_PS_ON_BY_DEFAULT; +@@ -570,9 +573,6 @@ use_default_name: + return NULL; + } + +- INIT_WORK(&rdev->wiphy_work, cfg80211_wiphy_work); +- INIT_LIST_HEAD(&rdev->wiphy_work_list); +- spin_lock_init(&rdev->wiphy_work_lock); + INIT_WORK(&rdev->rfkill_block, cfg80211_rfkill_block_work); + INIT_WORK(&rdev->conn_work, cfg80211_conn_work); + INIT_WORK(&rdev->event_work, cfg80211_event_work); |