diff options
| author | Timothy Pearson <tpearson@raptorengineeringinc.com> | 2015-06-20 14:40:56 -0500 |
|---|---|---|
| committer | Ronald G. Minnich <rminnich@gmail.com> | 2015-11-11 20:53:33 +0100 |
| commit | 9b5480dd5e6b2f84b4c05e9ce91a35cee4f9cc64 (patch) | |
| tree | e23448037ad3a4bc91034702d274d2b9b4b5d845 /src/northbridge/amd | |
| parent | fe2ae6190680db7999ba41df682dcfc59aa60322 (diff) | |
| download | coreboot-9b5480dd5e6b2f84b4c05e9ce91a35cee4f9cc64.tar.xz | |
northbridge/amd/amdmct: Fix crash on startup due to NULL pointer access
Change-Id: I47089f2ad886a6fda4e0cd4472efd975bb8e06c5
Signed-off-by: Timothy Pearson <tpearson@raptorengineeringinc.com>
Reviewed-on: http://review.coreboot.org/11995
Reviewed-by: Edward O'Callaghan <edward.ocallaghan@koparo.com>
Tested-by: build bot (Jenkins)
Diffstat (limited to 'src/northbridge/amd')
| -rw-r--r-- | src/northbridge/amd/amdmct/wrappers/mcti_d.c | 5 |
1 files changed, 2 insertions, 3 deletions
diff --git a/src/northbridge/amd/amdmct/wrappers/mcti_d.c b/src/northbridge/amd/amdmct/wrappers/mcti_d.c index fecda0b24d..45f817791b 100644 --- a/src/northbridge/amd/amdmct/wrappers/mcti_d.c +++ b/src/northbridge/amd/amdmct/wrappers/mcti_d.c @@ -341,11 +341,10 @@ 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 < MAX_DIMMS_SUPPORTED; dimm++) { - if (pDCTData->DimmRanks[dimm] > highest_rank_count[i]) - highest_rank_count[i] = pDCTData->DimmRanks[dimm]; + if (pDCTstat->DimmRanks[dimm] > highest_rank_count[i]) + highest_rank_count[i] = pDCTstat->DimmRanks[dimm]; } } #endif |