]> git.ipfire.org Git - thirdparty/kernel/linux.git/commit
net: pse-pd: fix out-of-bounds bitmap access in pse_isr() on 32-bit
authorKory Maincent <kory.maincent@bootlin.com>
Wed, 15 Apr 2026 13:02:59 +0000 (15:02 +0200)
committerJakub Kicinski <kuba@kernel.org>
Fri, 17 Apr 2026 02:25:32 +0000 (19:25 -0700)
commit5099807f335ce4f783f0578bef7278fffad30b07
tree132dafb924e59b506a51ecc14384b642a3c7dd10
parent0f99e0c3e19badaf3fdced0d3feba623e59eed41
net: pse-pd: fix out-of-bounds bitmap access in pse_isr() on 32-bit

In pse_isr(), notifs_mask was declared as a single unsigned long on the
stack (32 bits on 32-bit architectures). For PSE controllers with more
than 32 ports, this causes two problems:

- map_event callbacks could wrote bit positions >= 32 via
  *notifs_mask |= BIT(i), which is undefined behaviour on a 32-bit
  unsigned long and corrupts adjacent stack memory.

- for_each_set_bit(i, &notifs_mask, pcdev->nr_lines) treats
  &notifs_mask as a multi-word bitmap and reads beyond the single
  unsigned long when nr_lines > BITS_PER_LONG.

Fix this by moving notifs_mask out of the stack and into struct pse_irq
as a dynamically allocated bitmap. It is sized with
BITS_TO_LONGS(pcdev->nr_lines) words in devm_pse_irq_helper(), so it
is always wide enough regardless of the host word size.

[Jakub]: No upstream driver currently supports >=32 ports.

Signed-off-by: Kory Maincent <kory.maincent@bootlin.com>
Link: https://patch.msgid.link/20260415130300.806152-1-kory.maincent@bootlin.com
Signed-off-by: Jakub Kicinski <kuba@kernel.org>
drivers/net/pse-pd/pse_core.c