diff options
author | Changbin Du <changbin.du@intel.com> | 2017-11-08 01:26:53 +0100 |
---|---|---|
committer | Bjorn Helgaas <bhelgaas@google.com> | 2017-11-08 21:47:07 +0100 |
commit | a405f191f42ead45a03c000110a16683d30f7333 (patch) | |
tree | b215842a21055c1e8ccc854b3b5de7d6f8a4f84c /drivers/pci/rom.c | |
parent | PCI: Move pci_map_rom() error path (diff) | |
download | linux-a405f191f42ead45a03c000110a16683d30f7333.tar.xz linux-a405f191f42ead45a03c000110a16683d30f7333.zip |
PCI: Fail pci_map_rom() if the option ROM is invalid
If we detect a invalid PCI option ROM (e.g., invalid ROM header signature),
we should unmap it immediately and fail. It doesn't make any sense to
return a mapped area with size of 0.
I have seen this case on Intel GVTg vGPU, which has no VBIOS. It will not
cause a real problem, but we should skip it as early as possible.
Signed-off-by: Changbin Du <changbin.du@intel.com>
[bhelgaas: split non-functional change into separate patch]
Signed-off-by: Bjorn Helgaas <bhelgaas@google.com>
Diffstat (limited to 'drivers/pci/rom.c')
-rw-r--r-- | drivers/pci/rom.c | 5 |
1 files changed, 5 insertions, 0 deletions
diff --git a/drivers/pci/rom.c b/drivers/pci/rom.c index 0decf0d81cf5..1f5e6af96c83 100644 --- a/drivers/pci/rom.c +++ b/drivers/pci/rom.c @@ -156,8 +156,13 @@ void __iomem *pci_map_rom(struct pci_dev *pdev, size_t *size) * True size is important if the ROM is going to be copied. */ *size = pci_get_rom_size(pdev, rom, *size); + if (!*size) + goto invalid_rom; + return rom; +invalid_rom: + iounmap(rom); err_ioremap: /* restore enable if ioremap fails */ if (!(res->flags & IORESOURCE_ROM_ENABLE)) |