fbdev: fix cea_modes array size
authorTomi Valkeinen <tomi.valkeinen@ti.com>
Thu, 15 Jan 2015 11:47:19 +0000 (13:47 +0200)
committerTomi Valkeinen <tomi.valkeinen@ti.com>
Thu, 20 Aug 2015 07:20:11 +0000 (10:20 +0300)
commit6080a89357cc46f3450839a84af75c3d18f57772
tree9501a62b245f3dac2bd386d1e6e75b4879231254
parent2c6625cd545bdd66acff14f3394865d43920a5c7
fbdev: fix cea_modes array size

CEA defines 64 modes, indexed from 1 to 64. modedb has cea_modes arrays,
which contains 64 entries. However, the code uses the CEA indices
directly, i.e. the first mode is at cea_modes[1]. This means the array
is one too short.

This does not cause references to uninitialized memory as the code in
fbmon only allows indexes up to 63, and the cea_modes does not contain
an entry for the mode 64 so it could not be used in any case.

However, the code contains a check 'if (idx > ARRAY_SIZE(cea_modes)',
and while that check is a no-op as at that point idx cannot be >= 63, it
upsets static checkers.

Fix this by increasing the cea_array size to be 65, and change the code
to allow mode 64.

Signed-off-by: Tomi Valkeinen <tomi.valkeinen@ti.com>
Reported-by: Dan Carpenter <dan.carpenter@oracle.com>
drivers/video/fbdev/core/fbmon.c
drivers/video/fbdev/core/modedb.c
include/linux/fb.h
This page took 0.027603 seconds and 5 git commands to generate.