diff options
author | xypron.glpk@gmx.de <xypron.glpk@gmx.de> | 2017-07-30 21:15:51 +0300 |
---|---|---|
committer | Tom Rini <trini@konsulko.com> | 2017-08-13 22:17:26 +0300 |
commit | b69a0849e75e5adff081d1f0f5557da03d703482 (patch) | |
tree | 5dbba25d678264a26f744a52af4d777f7a4ee805 /arch/arm/cpu/armv7/bcm235xx | |
parent | 9730bcdc2f6c237fde66b7d81681e380ef0bdedf (diff) | |
download | u-boot-b69a0849e75e5adff081d1f0f5557da03d703482.tar.xz |
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>
Diffstat (limited to 'arch/arm/cpu/armv7/bcm235xx')
-rw-r--r-- | arch/arm/cpu/armv7/bcm235xx/clk-core.c | 2 |
1 files changed, 1 insertions, 1 deletions
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); |