projects / cascardo / linux.git / commit
? search:
summary | shortlog | log | commit | commitdiff | tree
(parent: cd70c37) | patch
[media] cec: fix off-by-one memset
authorHans Verkuil <hverkuil@xs4all.nl>
Fri, 22 Jul 2016 08:36:29 +0000 (05:36 -0300)
committerMauro Carvalho Chehab <mchehab@s-opensource.com>
Thu, 28 Jul 2016 23:16:35 +0000 (20:16 -0300)
commit292eaf50c7df4ae2ae8aaa9e1ce3f1240a353ee8
tree2f15beb3e5ff2268fda5c96b62916a2d3ee6701etree | snapshot
parentcd70c37b5a23eee7aa411205e2b4c74097f52336commit | diff
[media] cec: fix off-by-one memset

The unused bytes of the features array should be zeroed, but the start index was one
byte too early. This caused the device features byte to be overwritten by 0.

The compliance test for the CEC_S_LOG_ADDRS ioctl didn't catch this because it tested
byte continuation with the second device features byte being 0 :-(

Signed-off-by: Hans Verkuil <hans.verkuil@cisco.com>
Signed-off-by: Mauro Carvalho Chehab <mchehab@s-opensource.com>
drivers/staging/media/cec/cec-adap.c diff | blob | history
Unnamed repository; edit this file 'description' to name the repository.