diff options
author | Harry Morris <h.morris@cascoda.com> | 2018-03-28 12:54:27 +0200 |
---|---|---|
committer | Stefan Schmidt <stefan@osg.samsung.com> | 2018-03-29 16:51:26 +0200 |
commit | 86674a97f5055f4c7f406563408096e8cf9364ff (patch) | |
tree | ee99af9a7443c785412b163b350ba9e309e44760 /sound/ac97 | |
parent | ieee802154: remove unused variable 'val' (diff) | |
download | linux-86674a97f5055f4c7f406563408096e8cf9364ff.tar.xz linux-86674a97f5055f4c7f406563408096e8cf9364ff.zip |
ieee802154: ca8210: fix uninitialised data read
In ca8210_test_int_user_write() a user can request the transfer of a
frame with a length field (command.length) that is longer than the
actual buffer provided (len). In this scenario the driver will copy
the buffer contents into the uninitialised command[] buffer, then
transfer <data.length> bytes over the SPI even though only <len> bytes
had been populated, potentially leaking sensitive kernel memory.
Also the first 6 bytes of the command buffer must be initialised in case
a malformed, short packet is written and the uninitialised bytes are
read in ca8210_test_check_upstream.
Reported-by: Domen Puncer Kugler <domen.puncer@samsung.com>
Signed-off-by: Harry Morris <h.morris@cascoda.com>
Tested-by: Harry Morris <h.morris@cascoda.com>
Signed-off-by: Stefan Schmidt <stefan@osg.samsung.com>
Diffstat (limited to 'sound/ac97')
0 files changed, 0 insertions, 0 deletions