[coreboot-gerrit] New patch to review for coreboot: northbridge/amd/amdmct: Fix array indexing

[Jonathan Neuschäfer]

Jonathan Neuschäfer ([e-mail]) just uploaded a new patch set to gerrit, which you can find at https://review.coreboot.org/15553

-gerrit

commit 192628c42c7c1098dcaf75f030a7869e16c922fd
Author: Jonathan Neuschäfer 
Date:   Wed Jul 6 22:08:46 2016 +0200

northbridge/amd/amdmct: Fix array indexing

CH_D_BC_RCVRDLY is an array with dimensions [2][4]. The intention of the
    patched line was probably to point just behind the CH_D_BC_RCVRDLY field
    in struct DCTStatStruc, but this is achieved by using index [1][4] or
    [2][0].

Change-Id: Ia5da83cc369ca85ffc507a56a535698cbfa6bc39
    Reported-By: GCC 6.1.0
    Signed-off-by: Jonathan Neuschäfer 
---
 src/northbridge/amd/amdmct/mct/mct_d.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/northbridge/amd/amdmct/mct/mct_d.c b/src/northbridge/amd/amdmct/mct/mct_d.c
index 0e59e1d..c3b4589 100644
--- a/src/northbridge/amd/amdmct/mct/mct_d.c
+++ b/src/northbridge/amd/amdmct/mct/mct_d.c
@@ -3633,7 +3633,7 @@ static void mct_ResetDataStruct_D(struct MCTStatStruc *pMCTstat,
    		p[i] = 0;
    	}

-		start = (u32)(&((struct DCTStatStruc *)0)->CH_D_BC_RCVRDLY[2][4]);
+		start = (u32)(&((struct DCTStatStruc *)0)->CH_D_BC_RCVRDLY[1][4]);
    	stop = sizeof(struct DCTStatStruc);
    	for (i = start; i < stop; i++) {
    		p[i] = 0;