[coreboot-gerrit] Patch merged into coreboot/master: sis/sis966: don't store a 32bit value in a 16bit variable

gerrit at coreboot.org gerrit at coreboot.org
Sun Jul 31 19:19:41 CEST 2016


the following patch was just integrated into master:
commit 21ce6efe289ecc8eb6f047ceb3c3c25303e7272e
Author: Patrick Georgi <pgeorgi at chromium.org>
Date:   Fri Jul 29 18:19:46 2016 +0200

    sis/sis966: don't store a 32bit value in a 16bit variable
    
    That only makes deviceid == 0 (because shifting a 16bit value by 16
    bits, well...)
    
    Change-Id: Iddca1de20760f92f70fe2d05886b488e5b48313d
    Signed-off-by: Patrick Georgi <pgeorgi at chromium.org>
    Found-by: Coverity Scan #1229558
    Reviewed-on: https://review.coreboot.org/15960
    Tested-by: build bot (Jenkins)
    Reviewed-by: Paul Menzel <paulepanter at users.sourceforge.net>
    Reviewed-by: Duncan Laurie <dlaurie at chromium.org>


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

-gerrit



More information about the coreboot-gerrit mailing list