media: st-hva: Fix potential NULL pointer dereferences
[ Upstream commit b7fdd208687ba59ebfb09b2199596471c63b69e3 ] When ctx_id >= HVA_MAX_INSTANCES in hva_hw_its_irq_thread() it tries to access fields of ctx that is NULL at that point. The patch gets rid of these accesses. Found by Linux Driver Verification project (linuxtesting.org). Signed-off-by: Evgeny Novikov <novikov@ispras.ru> Signed-off-by: Hans Verkuil <hverkuil-cisco@xs4all.nl> Signed-off-by: Mauro Carvalho Chehab <mchehab+huawei@kernel.org> Signed-off-by: Sasha Levin <sashal@kernel.org>
This commit is contained in:

committed by
Greg Kroah-Hartman

parent
617afcee2a
commit
74ef2418a7
@@ -130,8 +130,7 @@ static irqreturn_t hva_hw_its_irq_thread(int irq, void *arg)
|
|||||||
ctx_id = (hva->sts_reg & 0xFF00) >> 8;
|
ctx_id = (hva->sts_reg & 0xFF00) >> 8;
|
||||||
if (ctx_id >= HVA_MAX_INSTANCES) {
|
if (ctx_id >= HVA_MAX_INSTANCES) {
|
||||||
dev_err(dev, "%s %s: bad context identifier: %d\n",
|
dev_err(dev, "%s %s: bad context identifier: %d\n",
|
||||||
ctx->name, __func__, ctx_id);
|
HVA_PREFIX, __func__, ctx_id);
|
||||||
ctx->hw_err = true;
|
|
||||||
goto out;
|
goto out;
|
||||||
}
|
}
|
||||||
|
|
||||||
|
Reference in New Issue
Block a user