[coreboot-gerrit] New patch to review for coreboot: northbridge/amd/amdmct: Fix hang on boot due to invalid array access

[Timothy Pearson]

Timothy Pearson ([e-mail]) just uploaded a new patch set to gerrit, which you can find at http://review.coreboot.org/11989

-gerrit

commit 552438fb40f830f356b12fb2e0bba4963e4149d7
Author: Timothy Pearson 
Date:   Fri Jun 12 19:43:06 2015 -0500

northbridge/amd/amdmct: Fix hang on boot due to invalid array access

Change-Id: I47755caf7d2ff59463c817e739f9cb2ddd367c18
    Signed-off-by: Timothy Pearson 
---
 src/northbridge/amd/amdmct/wrappers/mcti_d.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/src/northbridge/amd/amdmct/wrappers/mcti_d.c b/src/northbridge/amd/amdmct/wrappers/mcti_d.c
index 1a4e984..d1da07b 100644
--- a/src/northbridge/amd/amdmct/wrappers/mcti_d.c
+++ b/src/northbridge/amd/amdmct/wrappers/mcti_d.c
@@ -344,7 +344,7 @@ static void mctGet_MaxLoadFreq(struct DCTStatStruc *pDCTstat)
    }

#if (CONFIG_DIMM_SUPPORT & 0x000F)==0x0005 /* AMD_FAM10_DDR3 */
-	for (i = 0; i < 15; i = i + 2) {
+	for (i = 0; i < MAX_DIMMS_SUPPORTED; i = i + 2) {
    	if (pDCTstat->DIMMValid & (1 << i))
    		ch1_voltage |= pDCTstat->DimmConfiguredVoltage[i];
    	if (pDCTstat->DIMMValid & (1 << (i + 1)))
@@ -355,7 +355,7 @@ static void mctGet_MaxLoadFreq(struct DCTStatStruc *pDCTstat)
    for (i = 0; i < 2; i++) {
    	sDCTStruct *pDCTData = pDCTstat->C_DCTPtr[i];
    	highest_rank_count[i] = 0x0;
-		for (dimm = 0; dimm < 8; dimm++) {
+		for (dimm = 0; dimm < MAX_DIMMS_SUPPORTED; dimm++) {
    		if (pDCTData->DimmRanks[dimm] > highest_rank_count[i])
    			highest_rank_count[i] = pDCTData->DimmRanks[dimm];
    	}