[coreboot-gerrit] Patch merged into coreboot/master: soc/mediatek/mt8173/spi.c: Change assert to if statement

gerrit at coreboot.org gerrit at coreboot.org
Thu Nov 24 00:12:37 CET 2016


the following patch was just integrated into master:
commit ba1730b181ed1392d9c6b7ada7ab3bc35b68ffbd
Author: Martin Roth <martinroth at google.com>
Date:   Fri Nov 18 09:54:23 2016 -0700

    soc/mediatek/mt8173/spi.c: Change assert to if statement
    
    Asserts are only fatal if CONFIG_FATAL_ASSERTS is enabled in Kconfig.
    By default this is disabled, so the assert is generally just a printf.
    
    Die if someone decides to pass in an invalid bus number for some reason.
    
    Addresses coverity issue 1349858 - Out-of-bounds read
    
    Signed-off-by: Martin Roth <martinroth at google.com>
    Change-Id: I9d79bc336cbbfde31f655cfd271f101e7a90ab1b
    Reviewed-on: https://review.coreboot.org/17484
    Reviewed-by: Nico Huber <nico.h at gmx.de>


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

-gerrit



More information about the coreboot-gerrit mailing list