[coreboot-gerrit] Patch merged into coreboot/master: cpu/ti/am355x: Fix array overrun

gerrit at coreboot.org gerrit at coreboot.org
Sun Aug 14 19:07:24 CEST 2016


the following patch was just integrated into master:
commit 289f0578ce86bac8b0d03efa1244d0b0f8920364
Author: Paul Menzel <pmenzel at molgen.mpg.de>
Date:   Fri Aug 12 14:08:49 2016 +0200

    cpu/ti/am355x: Fix array overrun
    
    > Overrunning array "am335x_gpio_banks" of 4 4-byte elements at element
    > index 4 (byte offset 16) using index "bank" (which evaluates to 4).
    
    As the first index is 0, also error out if the index is equal the array
    size.
    
    Change-Id: I6b6b6e010348a58931bd546dfc54f08460e8dbbc
    Found-by: Coverity (CID 1354615:  Memory - illegal accesses  (OVERRUN))
    Signed-off-by: Paul Menzel <pmenzel at molgen.mpg.de>
    Reviewed-on: https://review.coreboot.org/16165
    Tested-by: build bot (Jenkins)
    Reviewed-by: Patrick Georgi <pgeorgi at google.com>


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

-gerrit



More information about the coreboot-gerrit mailing list