[coreboot-gerrit] New patch to review for coreboot: 9c30a0d northbridge/via/vx800: Fix out-of-bounds read due to off-by-one
Edward O'Callaghan (eocallaghan@alterapraxis.com)
gerrit at coreboot.org
Mon Aug 4 02:01:07 CEST 2014
Edward O'Callaghan (eocallaghan at alterapraxis.com) just uploaded a new patch set to gerrit, which you can find at http://review.coreboot.org/6485
-gerrit
commit 9c30a0d4342899453781560a3afe6b72e3e4a375
Author: Edward O'Callaghan <eocallaghan at alterapraxis.com>
Date: Mon Aug 4 09:58:30 2014 +1000
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 at alterapraxis.com>
---
src/northbridge/via/vx800/driving_setting.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/src/northbridge/via/vx800/driving_setting.c b/src/northbridge/via/vx800/driving_setting.c
index 695fcab..0c8865a 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;
More information about the coreboot-gerrit
mailing list