diff options
author | xypron.glpk@gmx.de <xypron.glpk@gmx.de> | 2017-07-30 21:22:47 +0300 |
---|---|---|
committer | Tom Rini <trini@konsulko.com> | 2017-08-13 22:17:27 +0300 |
commit | 65e8ce29e4efc7f1716828ec845bd294a942e1ca (patch) | |
tree | f62e4afd612fd7ef7fa81eabe07398c8b3e0bab3 /arch | |
parent | b69a0849e75e5adff081d1f0f5557da03d703482 (diff) | |
download | u-boot-65e8ce29e4efc7f1716828ec845bd294a942e1ca.tar.xz |
arm: bcm235xx: clk_set_rate avoid possible NULL deref
It does not make sense first to dereference c and then
to check if it is NULL.
Signed-off-by: Heinrich Schuchardt <xypron.glpk@gmx.de>
Diffstat (limited to 'arch')
-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 ee4b34574a..89e367be82 100644 --- a/arch/arm/cpu/armv7/bcm235xx/clk-core.c +++ b/arch/arm/cpu/armv7/bcm235xx/clk-core.c @@ -493,9 +493,9 @@ int clk_set_rate(struct clk *c, unsigned long rate) { int ret; - debug("%s: %s rate=%ld\n", __func__, c->name, rate); if (!c || !c->ops || !c->ops->set_rate) return -EINVAL; + debug("%s: %s rate=%ld\n", __func__, c->name, rate); if (c->use_cnt) return -EINVAL; |