of_match_device could return NULL, and so can cause a NULL pointer dereference later. Signed-off-by: Shailendra Verma <shailendra.v@xxxxxxxxxxx> --- drivers/pci/host/pcie-hisi.c | 4 ++++ 1 file changed, 4 insertions(+) diff --git a/drivers/pci/host/pcie-hisi.c b/drivers/pci/host/pcie-hisi.c index 56154c2..3256f8f 100644 --- a/drivers/pci/host/pcie-hisi.c +++ b/drivers/pci/host/pcie-hisi.c @@ -174,6 +174,10 @@ static int hisi_pcie_probe(struct platform_device *pdev) driver = dev->driver; match = of_match_device(driver->of_match_table, dev); + if (!match) { + dev_err(dev, "Error: No device match found\n"); + return -ENODEV; + } hisi_pcie->soc_ops = (struct pcie_soc_ops *) match->data; hisi_pcie->subctrl = -- 1.7.9.5 -- To unsubscribe from this list: send the line "unsubscribe linux-pci" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html