| Message ID | 1519866685-26271-1-git-send-email-shawn.lin@rock-chips.com |
|---|---|
| State | Superseded |
| Headers | show |
| Series | [v2] PCI: Fix possible NULL pointer dereference for of_pci_bus_find_domain_nr | expand |
diff --git a/drivers/pci/pci.c b/drivers/pci/pci.c index ae654e2..06f7f9d 100644 --- a/drivers/pci/pci.c +++ b/drivers/pci/pci.c @@ -5607,8 +5607,9 @@ static int of_pci_bus_find_domain_nr(struct device *parent) use_dt_domains = 0; domain = pci_get_new_domain_nr(); } else { - dev_err(parent, "Node %pOF has inconsistent \"linux,pci-domain\" property in DT\n", - parent->of_node); + if (parent) + pr_err("Node %pOF has ", parent->of_node) + pr_err("Inconsistent \"linux,pci-domain\" property in DT\n"); domain = -1; }
If the "parent" pointer passed to of_pci_bus_find_domain_nr() is NULL, don't dereference it. Signed-off-by: Shawn Lin <shawn.lin@rock-chips.com> --- Changes in v2: - Convert to use pr_err and only print valid device node if 'parent' isn't NULL. drivers/pci/pci.c | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-)