]> git.baikalelectronics.ru Git - kernel.git/commit
iio: health: afe4403: Fix oob read in afe4403_read_raw
authorWei Yongjun <weiyongjun1@huawei.com>
Mon, 7 Nov 2022 15:19:46 +0000 (15:19 +0000)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Thu, 8 Dec 2022 10:23:02 +0000 (11:23 +0100)
commitadcc69e7fcc7aee98ebc97b32ecfacf5704b703f
tree1b8caafa4e4e338176d11e3d14658d59e826787b
parent3f558edd780a499d2fdf37270a6e41ec0caa6409
iio: health: afe4403: Fix oob read in afe4403_read_raw

[ Upstream commit debfeb59a18779b89bfdc481f6f03746f07b6b88 ]

KASAN report out-of-bounds read as follows:

BUG: KASAN: global-out-of-bounds in afe4403_read_raw+0x42e/0x4c0
Read of size 4 at addr ffffffffc02ac638 by task cat/279

Call Trace:
 afe4403_read_raw
 iio_read_channel_info
 dev_attr_show

The buggy address belongs to the variable:
 afe4403_channel_leds+0x18/0xffffffffffffe9e0

This issue can be reproduced by singe command:

 $ cat /sys/bus/spi/devices/spi0.0/iio\:device0/in_intensity6_raw

The array size of afe4403_channel_leds is less than channels, so access
with chan->address cause OOB read in afe4403_read_raw. Fix it by moving
access before use it.

Fixes: fbd956c6130a ("iio: health/afe440x: Use regmap fields")
Signed-off-by: Wei Yongjun <weiyongjun1@huawei.com>
Acked-by: Andrew Davis <afd@ti.com>
Link: https://lore.kernel.org/r/20221107151946.89260-1-weiyongjun@huaweicloud.com
Signed-off-by: Jonathan Cameron <Jonathan.Cameron@huawei.com>
Signed-off-by: Sasha Levin <sashal@kernel.org>
drivers/iio/health/afe4403.c