Skip to content

Commit ac5a9bb

Browse files
committed
Merge tag 'irq_urgent_for_v5.17_rc2' of git://git.kernel.org/pub/scm/linux/kernel/git/tip/tip
Pull irq fix from Borislav Petkov: "A single use-after-free fix in the PCI MSI irq domain allocation path" * tag 'irq_urgent_for_v5.17_rc2' of git://git.kernel.org/pub/scm/linux/kernel/git/tip/tip: PCI/MSI: Prevent UAF in error path
2 parents 10c64a0 + a0af3d1 commit ac5a9bb

File tree

2 files changed

+2
-3
lines changed

2 files changed

+2
-3
lines changed

drivers/pci/msi/irqdomain.c

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -28,6 +28,7 @@ void pci_msi_teardown_msi_irqs(struct pci_dev *dev)
2828
msi_domain_free_irqs_descs_locked(domain, &dev->dev);
2929
else
3030
pci_msi_legacy_teardown_msi_irqs(dev);
31+
msi_free_msi_descs(&dev->dev);
3132
}
3233

3334
/**
@@ -171,8 +172,7 @@ struct irq_domain *pci_msi_create_irq_domain(struct fwnode_handle *fwnode,
171172
if (info->flags & MSI_FLAG_USE_DEF_CHIP_OPS)
172173
pci_msi_domain_update_chip_ops(info);
173174

174-
info->flags |= MSI_FLAG_ACTIVATE_EARLY | MSI_FLAG_DEV_SYSFS |
175-
MSI_FLAG_FREE_MSI_DESCS;
175+
info->flags |= MSI_FLAG_ACTIVATE_EARLY | MSI_FLAG_DEV_SYSFS;
176176
if (IS_ENABLED(CONFIG_GENERIC_IRQ_RESERVATION_MODE))
177177
info->flags |= MSI_FLAG_MUST_REACTIVATE;
178178

drivers/pci/msi/legacy.c

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -77,5 +77,4 @@ void pci_msi_legacy_teardown_msi_irqs(struct pci_dev *dev)
7777
{
7878
msi_device_destroy_sysfs(&dev->dev);
7979
arch_teardown_msi_irqs(dev);
80-
msi_free_msi_descs(&dev->dev);
8180
}

0 commit comments

Comments
 (0)