[coreboot-gerrit] Patch merged into coreboot/master: rtc: Do checksum check for all bytes

gerrit at coreboot.org gerrit at coreboot.org
Fri May 6 19:03:51 CEST 2016


the following patch was just integrated into master:
commit 24850ccf9b4597a1269ac3409814219108fadf33
Author: Stefan Reinauer <stefan.reinauer at coreboot.org>
Date:   Wed May 4 16:16:47 2016 -0700

    rtc: Do checksum check for all bytes
    
    Due to missing braces (that went undetected because of the
    indentation), I584189d9fcf7c9b831d9c020ee7ed59bb5ae08e8
    CMOS: add set_option() only takes the last changed byte into regard
    when determining whether the checksum needs to be updated.
    
    This bug went undetected for 5 years.
    
    Change-Id: I47cedc801a60959386dfdcda3a13b8e3162a7ecb
    Signed-off-by: Stefan Reinauer <stefan.reinauer at coreboot.org>
    Reviewed-on: https://review.coreboot.org/14616
    Tested-by: build bot (Jenkins)
    Reviewed-by: Paul Menzel <paulepanter at users.sourceforge.net>
    Reviewed-by: Martin Roth <martinroth at google.com>


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

-gerrit



More information about the coreboot-gerrit mailing list