[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