diff options
author | Jaehoon Chung <jh80.chung@samsung.com> | 2018-01-21 21:28:54 -0500 |
---|---|---|
committer | Lorenzo Pieralisi <lorenzo.pieralisi@arm.com> | 2018-01-23 09:37:11 -0500 |
commit | b5d6bc90c9129279d363ccbc02ad11e7b657c0b4 (patch) | |
tree | 8354e4db4267fc376cff69ee6ff2b42fbbebae59 | |
parent | 3b65ca50d24ce33cb92d88840e289135c92b40ed (diff) |
PCI: exynos: Fix a potential init_clk_resources NULL pointer dereference
In order to avoid triggering a NULL pointer dereference in
exynos_pcie_probe() a check must be put in place to detect if
the init_clk_resources hook is initialized before calling it.
Add the respective function pointer check in exynos_pcie_probe().
Signed-off-by: Jaehoon Chung <jh80.chung@samsung.com>
[lorenzo.pieralisi@arm.com: rewrote the commit log]
Signed-off-by: Lorenzo Pieralisi <lorenzo.pieralisi@arm.com>
-rw-r--r-- | drivers/pci/dwc/pci-exynos.c | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/drivers/pci/dwc/pci-exynos.c b/drivers/pci/dwc/pci-exynos.c index 56f32aeebd0a..39f08881f9a4 100644 --- a/drivers/pci/dwc/pci-exynos.c +++ b/drivers/pci/dwc/pci-exynos.c | |||
@@ -499,7 +499,8 @@ static int __init exynos_pcie_probe(struct platform_device *pdev) | |||
499 | return ret; | 499 | return ret; |
500 | } | 500 | } |
501 | 501 | ||
502 | if (ep->ops && ep->ops->get_clk_resources) { | 502 | if (ep->ops && ep->ops->get_clk_resources && |
503 | ep->ops->init_clk_resources) { | ||
503 | ret = ep->ops->get_clk_resources(ep); | 504 | ret = ep->ops->get_clk_resources(ep); |
504 | if (ret) | 505 | if (ret) |
505 | return ret; | 506 | return ret; |