diff options
| author | Paul Barker <paul.barker.ct@bp.renesas.com> | 2023-08-18 14:17:21 +0100 |
|---|---|---|
| committer | Marek Vasut <marek.vasut+renesas@mailbox.org> | 2023-08-19 16:34:41 +0200 |
| commit | dbada49554edd16da39134815635fd0015725c9e (patch) | |
| tree | 6692b32bf8823ff719e1fe62c6d91d32b1c7c377 /arch | |
| parent | bb89d926fc66fc754915f0300c77815e7db9327f (diff) | |
| download | u-boot-dbada49554edd16da39134815635fd0015725c9e.tar.gz u-boot-dbada49554edd16da39134815635fd0015725c9e.tar.bz2 u-boot-dbada49554edd16da39134815635fd0015725c9e.zip | |
arm: rmobile: Fix off-by-one error in cpuinfo
In rmobile_cpuinfo_idx() there is an off-by-one error in accessing the
rmobile_cpuinfo array.
At the end of the loop, i is equal to the array size, i.e.
rmobile_cpuinfo[i] accesses one entry past the end of the array. The
last entry in the array is a fallback value so the loop should count to
ARRAY_SIZE(rmobile_cpuinfo) - 1 instead, this will leave i equal to the
index of the fallback value if no match is found.
Signed-off-by: Paul Barker <paul.barker.ct@bp.renesas.com>
Reviewed-by: Biju Das <biju.das.jz@bp.renesas.com>
Reviewed-by: Marek Vasut <marek.vasut+renesas@mailbox.org>
Diffstat (limited to 'arch')
| -rw-r--r-- | arch/arm/mach-rmobile/cpu_info.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/arch/arm/mach-rmobile/cpu_info.c b/arch/arm/mach-rmobile/cpu_info.c index 1d33e2aa9d..7651e43bd0 100644 --- a/arch/arm/mach-rmobile/cpu_info.c +++ b/arch/arm/mach-rmobile/cpu_info.c @@ -86,7 +86,7 @@ static int rmobile_cpuinfo_idx(void) int i = 0; u32 cpu_type = rmobile_get_cpu_type(); - for (; i < ARRAY_SIZE(rmobile_cpuinfo); i++) + for (; i < ARRAY_SIZE(rmobile_cpuinfo) - 1; i++) if (rmobile_cpuinfo[i].cpu_type == cpu_type) break; |