diff options
author | Richard Spiegel <richard.spiegel@amd.corp-partner.google.com> | 2019-02-20 11:00:19 -0700 |
---|---|---|
committer | Patrick Georgi <pgeorgi@google.com> | 2019-02-28 13:57:07 +0000 |
commit | b28025a434cfb955a253a5b4d71e26e00f44a80d (patch) | |
tree | 6b43e65b3ee5af88cfbf9ee864d0b6f3adf29bf6 /src/soc/intel/braswell/acpi | |
parent | c60c3269ec9d1bfcec2ec4f6fb450e9a59a0818e (diff) | |
download | coreboot-b28025a434cfb955a253a5b4d71e26e00f44a80d.tar.xz |
cpu/x86/mtrr/mtrr.c:Avoid static scan false positive
Static scan does not know the contents of the fixed MTRR descriptor, so
it has no way to eval the result for variable num_ranges. If num_ranges
is less or equal to 0, the for loop will not be entered, and the values
of fixed_msrs will not be set. Asserting that num_ranges is greater than
0 ensures the loop enters at least once.
BUG=b:112253891
TEST=build grunt
Change-Id: Ieec0ac432c745bde4b1700539c266625da6cfd77
Signed-off-by: Richard Spiegel <richard.spiegel@silverbackltd.com>
Reviewed-on: https://review.coreboot.org/c/31527
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Reviewed-by: Martin Roth <martinroth@google.com>
Diffstat (limited to 'src/soc/intel/braswell/acpi')
0 files changed, 0 insertions, 0 deletions