diff options
| author | Anastasia Belova <[email protected]> | 2022-12-09 13:05:50 +0300 |
|---|---|---|
| committer | Thomas Bogendoerfer <[email protected]> | 2022-12-09 23:37:06 +0100 |
| commit | ee9ef11bd2a59c2fefaa0959e5efcdf040d7c654 (patch) | |
| tree | b9cb6772a1b1531021bc673aa71e21a2c9a4944c | |
| parent | 31495d4f6f1308d817a69f9228063a344c5c7ba4 (diff) | |
MIPS: BCM63xx: Add check for NULL for clk in clk_enable
Check clk for NULL before calling clk_enable_unlocked where clk
is dereferenced. There is such check in other implementations
of clk_enable.
Found by Linux Verification Center (linuxtesting.org) with SVACE.
Fixes: e7300d04bd08 ("MIPS: BCM63xx: Add support for the Broadcom BCM63xx family of SOCs.")
Signed-off-by: Anastasia Belova <[email protected]>
Reviewed-by: Philippe Mathieu-Daudé <[email protected]>
Acked-by: Florian Fainelli <[email protected]>
Signed-off-by: Thomas Bogendoerfer <[email protected]>
| -rw-r--r-- | arch/mips/bcm63xx/clk.c | 2 |
1 files changed, 2 insertions, 0 deletions
diff --git a/arch/mips/bcm63xx/clk.c b/arch/mips/bcm63xx/clk.c index 6e6756e8fa0a..86a6e2590866 100644 --- a/arch/mips/bcm63xx/clk.c +++ b/arch/mips/bcm63xx/clk.c @@ -361,6 +361,8 @@ static struct clk clk_periph = { */ int clk_enable(struct clk *clk) { + if (!clk) + return 0; mutex_lock(&clocks_mutex); clk_enable_unlocked(clk); mutex_unlock(&clocks_mutex); |