diff options
| author | Edward O'Callaghan <eocallaghan@alterapraxis.com> | 2014-08-04 09:58:30 +1000 |
|---|---|---|
| committer | Edward O'Callaghan <eocallaghan@alterapraxis.com> | 2014-08-05 02:12:20 +0200 |
| commit | 18fe07ed22e81c6315c199d5ae71b689163bac46 (patch) | |
| tree | b507f1b356b0ce6243b85bfe73c8d5e99f3da003 | |
| parent | 68a97164d3627bf277a7a5962e8006d163d40429 (diff) | |
| download | coreboot-18fe07ed22e81c6315c199d5ae71b689163bac46.tar.xz | |
northbridge/via/vx800: Fix out-of-bounds read due to off-by-one
Change-Id: Ia7fda59b60b2148dd4d246686bd94d2334b23eb5
Found-by: Coverity Scan
Signed-off-by: Edward O'Callaghan <eocallaghan@alterapraxis.com>
Reviewed-on: http://review.coreboot.org/6485
Tested-by: build bot (Jenkins)
Reviewed-by: Patrick Georgi <patrick@georgi-clan.de>
| -rw-r--r-- | src/northbridge/via/vx800/driving_setting.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/src/northbridge/via/vx800/driving_setting.c b/src/northbridge/via/vx800/driving_setting.c index 695fcabbf4..0c8865a2e6 100644 --- a/src/northbridge/via/vx800/driving_setting.c +++ b/src/northbridge/via/vx800/driving_setting.c @@ -370,7 +370,7 @@ void DrivingDCLK(DRAM_SYS_ATTR * DramAttr) else if (DIMMFREQ_667 == DramAttr->DramFreq) FreqId = 2; else if (DIMMFREQ_800 == DramAttr->DramFreq) - FreqId = 4; + FreqId = 3; else FreqId = 0; |