Skip to content

Commit 74956cf

Browse files
Suzuki K Poulosevijay-suman
authored andcommitted
irqchip/gic-v2m: Prevent use after free of gicv2m_get_fwnode()
[ Upstream commit 3318dc299b072a0511d6dfd8367f3304fb6d9827 ] With ACPI in place, gicv2m_get_fwnode() is registered with the pci subsystem as pci_msi_get_fwnode_cb(), which may get invoked at runtime during a PCI host bridge probe. But, the call back is wrongly marked as __init, causing it to be freed, while being registered with the PCI subsystem and could trigger: Unable to handle kernel paging request at virtual address ffff8000816c0400 gicv2m_get_fwnode+0x0/0x58 (P) pci_set_bus_msi_domain+0x74/0x88 pci_register_host_bridge+0x194/0x548 This is easily reproducible on a Juno board with ACPI boot. Retain the function for later use. Fixes: 0644b3d ("irqchip/gic-v2m: acpi: Introducing GICv2m ACPI support") Signed-off-by: Suzuki K Poulose <[email protected]> Signed-off-by: Thomas Gleixner <[email protected]> Signed-off-by: Ingo Molnar <[email protected]> Reviewed-by: Marc Zyngier <[email protected]> Cc: [email protected] Signed-off-by: Sasha Levin <[email protected]> (cherry picked from commit f95659affee301464f0d058d528d96b35b452da8) Signed-off-by: Vijayendra Suman <[email protected]>
1 parent 82b60a8 commit 74956cf

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed

drivers/irqchip/irq-gic-v2m.c

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -454,7 +454,7 @@ static int __init gicv2m_of_init(struct fwnode_handle *parent_handle,
454454
#ifdef CONFIG_ACPI
455455
static int acpi_num_msi;
456456

457-
static __init struct fwnode_handle *gicv2m_get_fwnode(struct device *dev)
457+
static struct fwnode_handle *gicv2m_get_fwnode(struct device *dev)
458458
{
459459
struct v2m_data *data;
460460

0 commit comments

Comments
 (0)