diff options
author | Colin Ian King <colin.king@canonical.com> | 2017-06-22 14:27:21 -0300 |
---|---|---|
committer | Mauro Carvalho Chehab <mchehab@s-opensource.com> | 2017-06-24 09:38:53 -0300 |
commit | bc8e2d627a5888c48d553abe534c341c1c096a5e (patch) | |
tree | 2c8bea73524611e498b639d851558c3c9d902159 /sound/aoa/soundbus/i2sbus | |
parent | 13c2bede49f996ca34a368dfe05be53d6b2ccc22 (diff) | |
download | linux-bc8e2d627a5888c48d553abe534c341c1c096a5e.tar.bz2 |
media: venus: fix loop wrap in cleanup of clks
The current pre-decrement is incorrect and should be replaced
with a post-decrement. Consider the case where the very first
clk_prepare_enable fails when i is 0; in this case the error
clean up will decrement the unsigned int which wraps to the
largest unsigned int value causing an array out of bounds read
on core->clks[i].
Detected by CoverityScan, CID#1446590 ("Out-of-bounds read")
Signed-off-by: Colin Ian King <colin.king@canonical.com>
Signed-off-by: Hans Verkuil <hansverk@cisco.com>
Signed-off-by: Mauro Carvalho Chehab <mchehab@s-opensource.com>
Diffstat (limited to 'sound/aoa/soundbus/i2sbus')
0 files changed, 0 insertions, 0 deletions