leandrof/u-boot
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

edid: Use sizeof() in cea_is_hdmi_vsdb_present()

Browse Source 
We should not use an open-coded value here. Use sizeof() instead.

Signed-off-by: Simon Glass <sjg@chromium.org>
Reported-by: Coverity (CID: 163252)
Fixes: 43c6bdd0 (edid: Add HDMI flag to timing info)
This commit is contained in:
Simon Glass 2017-06-07 10:28:39 -06:00 committed by Tom Rini
parent 1a596c44c0
commit 66a1b30d14
1 changed files with 2 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
common/edid.c
View File

@ -148,8 +148,8 @@ static bool cea_is_hdmi_vsdb_present(struct edid_cea861_info *info)
/* check for end of data block */
end = info->dtd_offset;
if (end == 0)
end = 127;
if (end < 4 || end > 127)
end = sizeof(info->data);
if (end < 4 || end > sizeof(info->data))
return false;
end -= 4;