Commit f68e68d2 authored Mar 01, 2018 by Dan Carpenter Committed by Mauro Carvalho Chehab Mar 06, 2018

media: ov5695: Off by one in ov5695_enum_frame_sizes()



The ">" should be ">=" so that we don't read one element beyond the end
of the array.

Fixes: 8a77009b ("media: ov5695: add support for OV5695 sensor")

Signed-off-by: Dan Carpenter <dan.carpenter@oracle.com>
Signed-off-by: Mauro Carvalho Chehab <mchehab@s-opensource.com>

parent 67f2219c

drivers/media/i2c/ov5695.c

+1 −1

Original line number	Diff line number	Diff line
		@@ -884,7 +884,7 @@ static int ov5695_enum_frame_sizes(struct v4l2_subdev *sd,
		struct v4l2_subdev_pad_config *cfg,
		struct v4l2_subdev_frame_size_enum *fse)
		{
		if (fse->index > ARRAY_SIZE(supported_modes))
		if (fse->index >= ARRAY_SIZE(supported_modes))
		return -EINVAL;

		if (fse->code != MEDIA_BUS_FMT_SBGGR10_1X10)