[coreboot-gerrit] Patch merged into coreboot/master: intel/common/opregion.c: only write 16 bytes to 16 byte field

gerrit at coreboot.org gerrit at coreboot.org
Thu Jul 28 19:22:42 CEST 2016


the following patch was just integrated into master:
commit 2098778726a209b3a9465b203a3fdb72c45706c0
Author: Martin Roth <martinroth at google.com>
Date:   Mon Jul 25 16:14:25 2016 -0600

    intel/common/opregion.c: only write 16 bytes to 16 byte field
    
    Including the terminating null, 17 characters were being written to the
    field, overwriting the a byte of the size field.
    
    Fortunately, the size was updated soon after this.
    
    Fixes coverity warning 1229570 - Destination buffer too small.
    
    Change-Id: I39285a9283dd9a17d638afe5b2755c7e420d7698
    Signed-off-by: Martin Roth <martinroth at google.com>
    Reviewed-on: https://review.coreboot.org/15889
    Tested-by: build bot (Jenkins)
    Reviewed-by: Aaron Durbin <adurbin at chromium.org>
    Reviewed-by: Paul Menzel <paulepanter at users.sourceforge.net>


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

-gerrit



More information about the coreboot-gerrit mailing list