Hello build bot (Jenkins),
I'd like you to reexamine a change. Please visit
https://review.coreboot.org/c/coreboot/+/38173
to look at the new patch set (#2).
Change subject: vc/amd/agesa/[...]/Config: Avoid out-of-bounds warnings ......................................................................
vc/amd/agesa/[...]/Config: Avoid out-of-bounds warnings
The memNTrainFlowControl array is generating Coverity warnings in multiple places in code where it attempts to write to index 1. The array is defined as either 2 elements or 1 of NULL depending on #if (AGESA_ENTRY_INIT_POST == TRUE). This is likely a false alarm from Coverity (memory should not be training outside of a POST), but adding a second NULL element for the AGESA_ENTRY_INIT_POST == FALSE case.
Change-Id: Ib56fb6d98f5afa0a73004c29d05647683e7a63d9 Signed-off-by: Joe Moore awokd@danwin1210.me Found-by: Coverity CID 1357451, 1357452, 1357453 --- M src/vendorcode/amd/agesa/f14/Config/OptionMemoryInstall.h M src/vendorcode/amd/agesa/f14/Proc/Mem/NB/mn.c M src/vendorcode/amd/agesa/f14/Proc/Mem/mn.h M src/vendorcode/amd/agesa/f15tn/Config/OptionMemoryInstall.h M src/vendorcode/amd/agesa/f15tn/Proc/Mem/NB/mn.c M src/vendorcode/amd/agesa/f15tn/Proc/Mem/mn.h M src/vendorcode/amd/agesa/f16kb/Config/OptionMemoryInstall.h 7 files changed, 14 insertions(+), 5 deletions(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/73/38173/2