[coreboot-gerrit] Patch merged into coreboot/master: 5cfef13 cpu/intel: Fix out-of-bounds read due to off-by-one in condition

gerrit at coreboot.org gerrit at coreboot.org
Mon Aug 4 13:58:15 CEST 2014


the following patch was just integrated into master:
commit 5cfef13f8d13b378f72b61ba3e4d7eee065f6d26
Author: Edward O'Callaghan <eocallaghan at alterapraxis.com>
Date:   Sun Aug 3 20:00:47 2014 +1000

    cpu/intel: Fix out-of-bounds read due to off-by-one in condition
    
    If power_limit_1_time > 129 is false then power_limit_1_time can have a
    value of up to 129 leading to an out-of-bounds illegal read indexing the
    power_limit_time_sec_to_msr[] array. Thankfully all call sites have been
    doing the right thing up until now so the issue has not been visible.
    
    Change-Id: Ic029d1af7fe43ca7da271043c2b08fe3088714af
    Found-by: Coverity Scan
    Signed-off-by: Edward O'Callaghan <eocallaghan at alterapraxis.com>
    Reviewed-on: http://review.coreboot.org/6478
    Tested-by: build bot (Jenkins)
    Reviewed-by: Patrick Georgi <patrick at georgi-clan.de>


See http://review.coreboot.org/6478 for details.

-gerrit



More information about the coreboot-gerrit mailing list