UbiquitousOS
/
OpenCloudOS-Kernel
mirror of https://gitee.com/OpenCloudOS/OpenCloudOS-Kernel.git
11
0
Fork 0
Code Issues Projects Releases Wiki Activity

media: saa7146: fix array overflow in vidioc_s_audio() 

The "a->index" value comes from the user via the ioctl.  The problem is
that the shift can wrap resulting in setting "mxb->cur_audinput" to an
invalid value, which later results in an array overflow.

Fixes: 6680427791 ("[media] mxb: fix audio handling")
Signed-off-by: Dan Carpenter <dan.carpenter@oracle.com>
Signed-off-by: Hans Verkuil <hverkuil-cisco@xs4all.nl>
Signed-off-by: Mauro Carvalho Chehab <mchehab+huawei@kernel.org>
This commit is contained in:
Dan Carpenter 2020-11-17 08:23:40 +01:00 committed by Mauro Carvalho Chehab
parent 28c1e371c2
commit 8e4d86e241
1 changed files with 10 additions and 9 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

7
drivers/media/pci/saa7146/mxb.c
View File

@ -641,7 +641,10 @@ static int vidioc_s_audio(struct file *file, void *fh, const struct v4l2_audio *
struct mxb *mxb = (struct mxb *)dev->ext_priv; struct mxb *mxb = (struct mxb *)dev->ext_priv;
DEB_D("VIDIOC_S_AUDIO %d\n", a->index); DEB_D("VIDIOC_S_AUDIO %d\n", a->index);
if (mxb_inputs[mxb->cur_input].audioset & (1 << a->index)) { if (a->index >= 32 ||
!(mxb_inputs[mxb->cur_input].audioset & (1 << a->index)))
return -EINVAL;
if (mxb->cur_audinput != a->index) { if (mxb->cur_audinput != a->index) {
mxb->cur_audinput = a->index; mxb->cur_audinput = a->index;
tea6420_route(mxb, a->index); tea6420_route(mxb, a->index);
@ -649,8 +652,6 @@ static int vidioc_s_audio(struct file *file, void *fh, const struct v4l2_audio *
mxb_update_audmode(mxb); mxb_update_audmode(mxb);
} }
return 0; return 0;
}
return -EINVAL;
} }
#ifdef CONFIG_VIDEO_ADV_DEBUG #ifdef CONFIG_VIDEO_ADV_DEBUG