media: s5p_cec: limit msg.len to CEC_MAX_MSG_SIZE
[ Upstream commit 93f65ce036
]
I expect that the hardware will have limited this to 16, but just in
case it hasn't, check for this corner case.
Signed-off-by: Hans Verkuil <hverkuil-cisco@xs4all.nl>
Signed-off-by: Mauro Carvalho Chehab <mchehab@kernel.org>
Signed-off-by: Sasha Levin <sashal@kernel.org>
This commit is contained in:
Родитель
647c12c47e
Коммит
cbfa26936f
|
@ -115,6 +115,8 @@ static irqreturn_t s5p_cec_irq_handler(int irq, void *priv)
|
||||||
dev_dbg(cec->dev, "Buffer overrun (worker did not process previous message)\n");
|
dev_dbg(cec->dev, "Buffer overrun (worker did not process previous message)\n");
|
||||||
cec->rx = STATE_BUSY;
|
cec->rx = STATE_BUSY;
|
||||||
cec->msg.len = status >> 24;
|
cec->msg.len = status >> 24;
|
||||||
|
if (cec->msg.len > CEC_MAX_MSG_SIZE)
|
||||||
|
cec->msg.len = CEC_MAX_MSG_SIZE;
|
||||||
cec->msg.rx_status = CEC_RX_STATUS_OK;
|
cec->msg.rx_status = CEC_RX_STATUS_OK;
|
||||||
s5p_cec_get_rx_buf(cec, cec->msg.len,
|
s5p_cec_get_rx_buf(cec, cec->msg.len,
|
||||||
cec->msg.msg);
|
cec->msg.msg);
|
||||||
|
|
Загрузка…
Ссылка в новой задаче