diff options
| author | Thomas Tai | 2018-07-26 19:13:04 +0200 |
|---|---|---|
| committer | Bjorn Helgaas | 2018-07-26 19:13:04 +0200 |
| commit | bd91b56cb3b27492963caeb5fccefe20a986ca8d (patch) | |
| tree | 3df6c7e55a6802db5af2b11791b52f120ed2ea5f /drivers/pci/pcie/err.c | |
| parent | PCI: v3-semi: Fix I/O space page leak (diff) | |
| download | kernel-qcow2-linux-bd91b56cb3b27492963caeb5fccefe20a986ca8d.tar.gz kernel-qcow2-linux-bd91b56cb3b27492963caeb5fccefe20a986ca8d.tar.xz kernel-qcow2-linux-bd91b56cb3b27492963caeb5fccefe20a986ca8d.zip | |
PCI/AER: Work around use-after-free in pcie_do_fatal_recovery()
When an fatal error is received by a non-bridge device, the device is
removed, and pci_stop_and_remove_bus_device() deallocates the device
structure. The freed device structure is used by subsequent code to send
uevents and print messages.
Hold a reference on the device until we're finished using it. This is not
an ideal fix because pcie_do_fatal_recovery() should not use the device at
all after removing it, but that's too big a project for right now.
Fixes: 7e9084b36740 ("PCI/AER: Handle ERR_FATAL with removal and re-enumeration of devices")
Signed-off-by: Thomas Tai <thomas.tai@oracle.com>
[bhelgaas: changelog, reduce get/put coverage]
Signed-off-by: Bjorn Helgaas <bhelgaas@google.com>
Diffstat (limited to 'drivers/pci/pcie/err.c')
| -rw-r--r-- | drivers/pci/pcie/err.c | 2 |
1 files changed, 2 insertions, 0 deletions
diff --git a/drivers/pci/pcie/err.c b/drivers/pci/pcie/err.c index f7ce0cb0b0b7..f02e334beb45 100644 --- a/drivers/pci/pcie/err.c +++ b/drivers/pci/pcie/err.c @@ -295,6 +295,7 @@ void pcie_do_fatal_recovery(struct pci_dev *dev, u32 service) parent = udev->subordinate; pci_lock_rescan_remove(); + pci_dev_get(dev); list_for_each_entry_safe_reverse(pdev, temp, &parent->devices, bus_list) { pci_dev_get(pdev); @@ -328,6 +329,7 @@ void pcie_do_fatal_recovery(struct pci_dev *dev, u32 service) pci_info(dev, "Device recovery from fatal error failed\n"); } + pci_dev_put(dev); pci_unlock_rescan_remove(); } |