diff options
| author | Dan Carpenter <dan.carpenter@oracle.com> | 2022-03-16 11:35:33 +0300 |
|---|---|---|
| committer | Stephen Boyd <sboyd@kernel.org> | 2022-03-17 12:21:28 -0700 |
| commit | c5601e0720ce1a3ad895f94a5838530edde01ed3 (patch) | |
| tree | f843def20384524c009457bf8aaea0190ad243a0 /kernel/utsname.c | |
| parent | e783362eb54cd99b2cac8b3a9aeac942e6f6ac07 (diff) | |
| download | linux-c5601e0720ce1a3ad895f94a5838530edde01ed3.tar.gz linux-c5601e0720ce1a3ad895f94a5838530edde01ed3.tar.bz2 linux-c5601e0720ce1a3ad895f94a5838530edde01ed3.zip | |
clk: visconti: prevent array overflow in visconti_clk_register_gates()
This code was using -1 to represent that there was no reset function.
Unfortunately, the -1 was stored in u8 so the if (clks[i].rs_id >= 0)
condition was always true. This lead to an out of bounds access in
visconti_clk_register_gates().
Fixes: b4cbe606dc36 ("clk: visconti: Add support common clock driver and reset driver")
Signed-off-by: Dan Carpenter <dan.carpenter@oracle.com>
Link: https://lore.kernel.org/r/20220316083533.GA30941@kili
Acked-by: Nobuhiro Iwamatsu <nobuhiro1.iwamatsu@toshiba.co.jp>
Signed-off-by: Stephen Boyd <sboyd@kernel.org>
Diffstat (limited to 'kernel/utsname.c')
0 files changed, 0 insertions, 0 deletions