[coreboot-gerrit] Patch merged into coreboot/master: soc/intel/fsp_baytrail: Make sure i2c bus is < 7

gerrit at coreboot.org gerrit at coreboot.org
Tue Dec 22 23:07:29 CET 2015


the following patch was just integrated into master:
commit c89099641ee2a8a33d9a92e11751915472e249ff
Author: Martin Roth <martinroth at google.com>
Date:   Mon Dec 21 13:14:58 2015 -0700

    soc/intel/fsp_baytrail: Make sure i2c bus is < 7
    
    Baytrail has I2c Busses 0 to 6, so is supposed to error out
    if the I2c driver is called with 7 or greater.  Due to an off-by-one
    error it could be called with bus 7.
    
    Fixes coverity warning:
    CID 1287074 (#1 of 1): Out-of-bounds read (OVERRUN)
    3. overrun-local: Overrunning array base_adr of 7 4-byte elements at
    element index 7 (byte offset 28) using index bus (which evaluates to 7).
    
    Change-Id: I7caec60298cf27bd669796e0e05e4a896f92befd
    Signed-off-by: Martin Roth <martinroth at google.com>
    Reviewed-on: https://review.coreboot.org/12781
    Tested-by: build bot (Jenkins)
    Reviewed-by: Werner Zeh <werner.zeh at siemens.com>


See https://review.coreboot.org/12781 for details.

-gerrit



More information about the coreboot-gerrit mailing list