ARM: EXYNOS: Handle of_find_device_by_node() and kstrdup() failures

Prevent possible NULL pointer dereference of pointer returned by
of_find_device_by_node(). Handle this by skipping such power domain.

Additionally fail the init on kstrdup() failure. Such case is actually
not fatal because the name for power domain allocated by kstrdup() is
used only in printk. Still as a precaution handle this as an error
condition.

Signed-off-by: Krzysztof Kozlowski <k.kozlowski@samsung.com>
Signed-off-by: Kukjin Kim <kgene@kernel.org>

diff --git a/arch/arm/mach-exynos/pm_domains.c b/arch/arm/mach-exynos/pm_domains.c
index b15f8663f9fc0e1351d53fd1897fa6d195ffea4f..3b54f6e8494cd0537a8c8ca0e0d977b45dbf2897 100644 (file)
--- a/arch/arm/mach-exynos/pm_domains.c
+++ b/arch/arm/mach-exynos/pm_domains.c
@@ -126,6 +126,12 @@ static __init int exynos4_pm_init_power_domain(void)
                struct device *dev;
 
                pdev = of_find_device_by_node(np);
+               if (!pdev) {
+                       pr_err("%s: failed to find device for node %s\n",
+                                       __func__, np->name);
+                       of_node_put(np);
+                       continue;
+               }
                dev = &pdev->dev;
 
                pd = kzalloc(sizeof(*pd), GFP_KERNEL);
@@ -136,6 +142,12 @@ static __init int exynos4_pm_init_power_domain(void)
                }
 
                pd->pd.name = kstrdup(dev_name(dev), GFP_KERNEL);
+               if (!pd->pd.name) {
+                       kfree(pd);
+                       of_node_put(np);
+                       return -ENOMEM;
+               }
+
                pd->name = pd->pd.name;
                pd->base = of_iomap(np, 0);
                if (!pd->base) {