media: v4l2-dv-timings.c: fix wrong condition in two for-loops
authorHans Verkuil <hverkuil-cisco@xs4all.nl>
Fri, 23 Jul 2021 08:22:59 +0000 (10:22 +0200)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Sat, 18 Sep 2021 11:40:24 +0000 (13:40 +0200)
commite3a2e20ed5037e668df4522575fd09a2b87263c3
tree0e708358b0ab31a23f5ed82aa1463be4eb09015d
parentac1bcf53e32fe503e0d311af4ae14789c6566210
media: v4l2-dv-timings.c: fix wrong condition in two for-loops

[ Upstream commit 4108b3e6db31acc4c68133290bbcc87d4db905c9 ]

These for-loops should test against v4l2_dv_timings_presets[i].bt.width,
not if i < v4l2_dv_timings_presets[i].bt.width. Luckily nothing ever broke,
since the smallest width is still a lot higher than the total number of
presets, but it is wrong.

The last item in the presets array is all 0, so the for-loop must stop
when it reaches that sentinel.

Signed-off-by: Hans Verkuil <hverkuil-cisco@xs4all.nl>
Reported-by: Krzysztof HaƂasa <khalasa@piap.pl>
Signed-off-by: Mauro Carvalho Chehab <mchehab+huawei@kernel.org>
Signed-off-by: Sasha Levin <sashal@kernel.org>
drivers/media/v4l2-core/v4l2-dv-timings.c