From: xypron.glpk@gmx.de Date: Sun, 30 Jul 2017 18:15:51 +0000 (+0200) Subject: arm: bcm235xx: avoid possible NULL dereference X-Git-Url: http://git.dujemihanovic.xyz/%22/img/sics.gif/%22/static/git-favicon.png?a=commitdiff_plain;h=b69a0849e75e5adff081d1f0f5557da03d703482;p=u-boot.git 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 --- 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);