From: xypron.glpk@gmx.de <xypron.glpk@gmx.de>
Date: Sun, 30 Jul 2017 18:15:51 +0000 (+0200)
Subject: arm: bcm235xx: avoid possible NULL dereference
X-Git-Tag: v2017.09-rc2~33
X-Git-Url: https://git.sur5r.net/?a=commitdiff_plain;h=b69a0849e75e5adff081d1f0f5557da03d703482;p=u-boot

arm: bcm235xx: avoid possible NULL dereference

It does not make sense to first dereference c and then
check if it is NULL.

Signed-off-by: Heinrich Schuchardt <xypron.glpk@gmx.de>
---

diff --git a/arch/arm/cpu/armv7/bcm235xx/clk-core.c b/arch/arm/cpu/armv7/bcm235xx/clk-core.c
index 79fafa08ed..ee4b34574a 100644
--- a/arch/arm/cpu/armv7/bcm235xx/clk-core.c
+++ b/arch/arm/cpu/armv7/bcm235xx/clk-core.c
@@ -479,9 +479,9 @@ unsigned long clk_get_rate(struct clk *c)
 {
 	unsigned long rate;
 
-	debug("%s: %s\n", __func__, c->name);
 	if (!c || !c->ops || !c->ops->get_rate)
 		return 0;
+	debug("%s: %s\n", __func__, c->name);
 
 	rate = c->ops->get_rate(c);
 	debug("%s: rate = %ld\n", __func__, rate);