diff options
author | Jacob Garber <jgarber1@ualberta.ca> | 2019-07-22 15:16:30 -0600 |
---|---|---|
committer | Patrick Georgi <pgeorgi@google.com> | 2019-07-30 09:56:02 +0000 |
commit | bcdb893778f857f310115522bbf7d70ad0cc017f (patch) | |
tree | 4a8e4314bdc7527f130bc78e172212cd054e54c0 /src/vendorcode | |
parent | 4926e989ac2f83bd887bee683c7e2c0481f5cd3a (diff) | |
download | coreboot-bcdb893778f857f310115522bbf7d70ad0cc017f.tar.xz |
soc/intel/{broad,cannon,sky}: Fix possible out-of-bounds reads
There will be a possible out of bounds array access if
power_limit_1_time == ARRAY_SIZE(power_limit_time_sec_to_msr), so
prevent that in the index check. This issue was fixed for other cpus in
commit 5cfef13f8d (cpu/intel: Fix out-of-bounds read due to off-by-one
in condition). Based on the discussion for that commit, also remove the
magic constant 28 in favour of the index of the last array element.
Change-Id: Ic3f8735b23a368f8a9395757bd52c2c40088afa1
Signed-off-by: Jacob Garber <jgarber1@ualberta.ca>
Found-by: Coverity CID 1229673
Reviewed-on: https://review.coreboot.org/c/coreboot/+/34498
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Reviewed-by: Patrick Rudolph <siro@das-labor.org>
Diffstat (limited to 'src/vendorcode')
0 files changed, 0 insertions, 0 deletions