]> git.baikalelectronics.ru Git - uboot.git/commit
IOMUX: Fix access past end of console_devices
authorSean Anderson <seanga2@gmail.com>
Wed, 6 Apr 2022 18:36:35 +0000 (14:36 -0400)
committerTom Rini <trini@konsulko.com>
Tue, 19 Apr 2022 18:52:34 +0000 (14:52 -0400)
commit0602d59b9d1a0fba9f90705f6c599e27c087b928
tree70b9a1d4af7cad7328f0c610657a4fc90e51d1f9
parentfacbd397a0f11934a8c7219ad10e61ad2a5988d4
IOMUX: Fix access past end of console_devices

We should only access console_devices[file][i] once we have checked that i
< cd_count[file]. Otherwise, we will access uninitialized memory at the end
of the loop. console_devices[file][i] should not be NULL, but putting the
assignment in the loop condition allows us to ensure that i is checked
beforehand. This isn't a bug, but it does make valgrind stop complaining.

Fixes: 1f7a7356d6 ("IOMUX: Split out for_each_console_dev() helper macro")
Signed-off-by: Sean Anderson <seanga2@gmail.com>
Reviewed-by: Andrew Scull <ascull@google.com>
Acked-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com>
include/iomux.h