[coreboot-gerrit] Change in ...coreboot[master]: soc/intel/common: Check variable bios_size after MIN operation

[John Zhao (Code Review)]

Hello Patrick Rudolph, Arthur Heymans, Balaji Manigandan, Lance Zhao, build bot (Jenkins), Martin Roth,

I'd like you to reexamine a change. Please visit

https://review.coreboot.org/c/coreboot/+/32924

to look at the new patch set (#3).

Change subject: soc/intel/common: Check variable bios_size after MIN operation
......................................................................

soc/intel/common: Check variable bios_size after MIN operation

Clang Static Analyzer version 8.0.0 found that the rounding operation
of alignment=1UL<<(log2_ceil(bios_size)) is garbage or undefined if
variable bios_size is zero. Move sanity check for bios_size after MIN
operation to prevent rounding error.

TEST=Built and boot up to kernel.

Change-Id: Ifc3f3da52d129ef5d6063a46b045603a236be759
Signed-off-by: John Zhao 
---
M src/soc/intel/common/block/fast_spi/fast_spi.c
1 file changed, 2 insertions(+), 3 deletions(-)

git pull ssh://review.coreboot.org:29418/coreboot refs/changes/24/32924/3