[coreboot-gerrit] Patch merged into coreboot/master: 477b4c5 AMD Fam10h: Don't write uninitialized data into ACPI

gerrit at coreboot.org gerrit at coreboot.org
Mon Feb 23 20:33:50 CET 2015


the following patch was just integrated into master:
commit 477b4c539a5c05a0ccdf11b68fdff78b2ebe56cb
Author: Patrick Georgi <patrick at georgi-clan.de>
Date:   Sun Feb 22 16:21:01 2015 +0100

    AMD Fam10h: Don't write uninitialized data into ACPI
    
    The goto statement skipped all the code that is necessary
    to fill in the data structures that are read right after
    the jump.
    
    Since there doesn't seem to be useful data, why write these
    ACPI objects in the first place?
    
    Change-Id: I1d06c11a7a31517b81e54159355d5c27e3cc3735
    Signed-off-by: Patrick Georgi <patrick at georgi-clan.de>
    Found-by: Coverity Scan
    Reviewed-on: http://review.coreboot.org/8507
    Tested-by: build bot (Jenkins)
    Reviewed-by: Paul Menzel <paulepanter at users.sourceforge.net>
    Reviewed-by: Timothy Pearson <tpearson at raptorengineeringinc.com>
    Reviewed-by: Alexandru Gagniuc <mr.nuke.me at gmail.com>


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

-gerrit



More information about the coreboot-gerrit mailing list