mirror of https://lore.kernel.org/lkml/
 help / color / mirror / Atom feed
* [PATCH AUTOSEL 6.6 01/16] wifi: mac80211: fix mbss changed flags corruption on 32 bit systems
@ 2024-12-20 17:12 Sasha Levin
  2024-12-20 17:12 ` [PATCH AUTOSEL 6.6 02/16] wifi: mac80211: wake the queues in case of failure in resume Sasha Levin
                   ` (14 more replies)
  0 siblings, 15 replies; 16+ messages in thread
From: Sasha Levin @ 2024-12-20 17:12 UTC (permalink / raw)
  To: linux-kernel, stable
  Cc: Issam Hamdi, Johannes Berg, Sasha Levin, johannes, linux-wireless

From: Issam Hamdi <ih@simonwunderlich.de>

[ Upstream commit 49dba1ded8dd5a6a12748631403240b2ab245c34 ]

On 32-bit systems, the size of an unsigned long is 4 bytes,
while a u64 is 8 bytes. Therefore, when using
or_each_set_bit(bit, &bits, sizeof(changed) * BITS_PER_BYTE),
the code is incorrectly searching for a bit in a 32-bit
variable that is expected to be 64 bits in size,
leading to incorrect bit finding.

Solution: Ensure that the size of the bits variable is correctly
adjusted for each architecture.

 Call Trace:
  ? show_regs+0x54/0x58
  ? __warn+0x6b/0xd4
  ? ieee80211_link_info_change_notify+0xcc/0xd4 [mac80211]
  ? report_bug+0x113/0x150
  ? exc_overflow+0x30/0x30
  ? handle_bug+0x27/0x44
  ? exc_invalid_op+0x18/0x50
  ? handle_exception+0xf6/0xf6
  ? exc_overflow+0x30/0x30
  ? ieee80211_link_info_change_notify+0xcc/0xd4 [mac80211]
  ? exc_overflow+0x30/0x30
  ? ieee80211_link_info_change_notify+0xcc/0xd4 [mac80211]
  ? ieee80211_mesh_work+0xff/0x260 [mac80211]
  ? cfg80211_wiphy_work+0x72/0x98 [cfg80211]
  ? process_one_work+0xf1/0x1fc
  ? worker_thread+0x2c0/0x3b4
  ? kthread+0xc7/0xf0
  ? mod_delayed_work_on+0x4c/0x4c
  ? kthread_complete_and_exit+0x14/0x14
  ? ret_from_fork+0x24/0x38
  ? kthread_complete_and_exit+0x14/0x14
  ? ret_from_fork_asm+0xf/0x14
  ? entry_INT80_32+0xf0/0xf0

Signed-off-by: Issam Hamdi <ih@simonwunderlich.de>
Link: https://patch.msgid.link/20241125162920.2711462-1-ih@simonwunderlich.de
[restore no-op path for no changes]
Signed-off-by: Johannes Berg <johannes.berg@intel.com>
Signed-off-by: Sasha Levin <sashal@kernel.org>
---
 net/mac80211/mesh.c | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/net/mac80211/mesh.c b/net/mac80211/mesh.c
index 25223184d6e5..a5e7edd2f2d1 100644
--- a/net/mac80211/mesh.c
+++ b/net/mac80211/mesh.c
@@ -1173,14 +1173,14 @@ void ieee80211_mbss_info_change_notify(struct ieee80211_sub_if_data *sdata,
 				       u64 changed)
 {
 	struct ieee80211_if_mesh *ifmsh = &sdata->u.mesh;
-	unsigned long bits = changed;
+	unsigned long bits[] = { BITMAP_FROM_U64(changed) };
 	u32 bit;
 
-	if (!bits)
+	if (!changed)
 		return;
 
 	/* if we race with running work, worst case this work becomes a noop */
-	for_each_set_bit(bit, &bits, sizeof(changed) * BITS_PER_BYTE)
+	for_each_set_bit(bit, bits, sizeof(changed) * BITS_PER_BYTE)
 		set_bit(bit, ifmsh->mbss_changed);
 	set_bit(MESH_WORK_MBSS_CHANGED, &ifmsh->wrkq_flags);
 	wiphy_work_queue(sdata->local->hw.wiphy, &sdata->work);
-- 
2.39.5


^ permalink raw reply	[flat|nested] 16+ messages in thread

end of thread, other threads:[~2024-12-20 17:13 UTC | newest]

Thread overview: 16+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2024-12-20 17:12 [PATCH AUTOSEL 6.6 01/16] wifi: mac80211: fix mbss changed flags corruption on 32 bit systems Sasha Levin
2024-12-20 17:12 ` [PATCH AUTOSEL 6.6 02/16] wifi: mac80211: wake the queues in case of failure in resume Sasha Levin
2024-12-20 17:12 ` [PATCH AUTOSEL 6.6 03/16] drm/amdkfd: Correct the migration DMA map direction Sasha Levin
2024-12-20 17:12 ` [PATCH AUTOSEL 6.6 04/16] btrfs: flush delalloc workers queue before stopping cleaner kthread during unmount Sasha Levin
2024-12-20 17:12 ` [PATCH AUTOSEL 6.6 05/16] ALSA: hda/ca0132: Use standard HD-audio quirk matching helpers Sasha Levin
2024-12-20 17:12 ` [PATCH AUTOSEL 6.6 06/16] ALSA: hda/realtek: Add new alc2xx-fixup-headset-mic model Sasha Levin
2024-12-20 17:12 ` [PATCH AUTOSEL 6.6 07/16] sound: usb: enable DSD output for ddHiFi TC44C Sasha Levin
2024-12-20 17:12 ` [PATCH AUTOSEL 6.6 08/16] sound: usb: format: don't warn that raw DSD is unsupported Sasha Levin
2024-12-20 17:12 ` [PATCH AUTOSEL 6.6 09/16] bpf: fix potential error return Sasha Levin
2024-12-20 17:12 ` [PATCH AUTOSEL 6.6 10/16] ksmbd: retry iterate_dir in smb2_query_dir Sasha Levin
2024-12-20 17:12 ` [PATCH AUTOSEL 6.6 11/16] ksmbd: set ATTR_CTIME flags when setting mtime Sasha Levin
2024-12-20 17:12 ` [PATCH AUTOSEL 6.6 12/16] smb: client: destroy cfid_put_wq on module exit Sasha Levin
2024-12-20 17:12 ` [PATCH AUTOSEL 6.6 13/16] net: usb: qmi_wwan: add Telit FE910C04 compositions Sasha Levin
2024-12-20 17:12 ` [PATCH AUTOSEL 6.6 14/16] Bluetooth: hci_core: Fix sleeping function called from invalid context Sasha Levin
2024-12-20 17:12 ` [PATCH AUTOSEL 6.6 15/16] irqchip/gic: Correct declaration of *percpu_base pointer in union gic_base Sasha Levin
2024-12-20 17:12 ` [PATCH AUTOSEL 6.6 16/16] ARC: build: Try to guess GCC variant of cross compiler Sasha Levin

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox

all inboxes | Powered by JetHome®