arm: bcm235xx: avoid possible NULL dereference
authorxypron.glpk@gmx.de <xypron.glpk@gmx.de>
Sun, 30 Jul 2017 18:15:51 +0000 (20:15 +0200)
committerTom Rini <trini@konsulko.com>
Sun, 13 Aug 2017 19:17:26 +0000 (15:17 -0400)
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>
arch/arm/cpu/armv7/bcm235xx/clk-core.c

index 79fafa08ed9358f3817658eff65159968ab85022..ee4b34574a0dc2436d6a0c69ffe3058ca07d2da4 100644 (file)
@@ -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);