diff options
author | Dmitry Osipenko <digetx@gmail.com> | 2022-01-12 22:50:39 +0300 |
---|---|---|
committer | Mark Brown <broonie@kernel.org> | 2022-01-28 18:59:19 +0300 |
commit | 06feec6005c9d9500cd286ec440aabf8b2ddd94d (patch) | |
tree | e69491266a7c1a6cf6e040a0f6f1afe8c2b0435d /include/uapi/sound/asound.h | |
parent | fb25621da5702c104ce0a48de5b174ced09e5b4e (diff) | |
download | linux-06feec6005c9d9500cd286ec440aabf8b2ddd94d.tar.xz |
ASoC: hdmi-codec: Fix OOB memory accesses
Correct size of iec_status array by changing it to the size of status
array of the struct snd_aes_iec958. This fixes out-of-bounds slab
read accesses made by memcpy() of the hdmi-codec driver. This problem
is reported by KASAN.
Cc: stable@vger.kernel.org
Signed-off-by: Dmitry Osipenko <digetx@gmail.com>
Link: https://lore.kernel.org/r/20220112195039.1329-1-digetx@gmail.com
Signed-off-by: Mark Brown <broonie@kernel.org>
Diffstat (limited to 'include/uapi/sound/asound.h')
-rw-r--r-- | include/uapi/sound/asound.h | 4 |
1 files changed, 3 insertions, 1 deletions
diff --git a/include/uapi/sound/asound.h b/include/uapi/sound/asound.h index ff7e638221c5..228279ea0670 100644 --- a/include/uapi/sound/asound.h +++ b/include/uapi/sound/asound.h @@ -56,8 +56,10 @@ * * ****************************************************************************/ +#define AES_IEC958_STATUS_SIZE 24 + struct snd_aes_iec958 { - unsigned char status[24]; /* AES/IEC958 channel status bits */ + unsigned char status[AES_IEC958_STATUS_SIZE]; /* AES/IEC958 channel status bits */ unsigned char subcode[147]; /* AES/IEC958 subcode bits */ unsigned char pad; /* nothing */ unsigned char dig_subframe[4]; /* AES/IEC958 subframe bits */ |