[coreboot-gerrit] Patch merged into coreboot/master: 801aa7c libpayload: fix a mips memmove() bug

gerrit at coreboot.org gerrit at coreboot.org
Wed Apr 22 08:50:46 CEST 2015


the following patch was just integrated into master:
commit 801aa7c355702efe2a6e505d59431980099a1f14
Author: Vadim Bendebury <vbendeb at chromium.org>
Date:   Thu Mar 26 13:35:23 2015 -0700

    libpayload: fix a mips memmove() bug
    
    size_t is an unsigned type and as such is a bad choice for a counting
    down loop counter.
    
    BRANCH=all
    BUG=none
    TEST=editing cli command line does not cause hangs any more
    
    Change-Id: I0502553b5e2143052345edeb205a01558fccd9b8
    Signed-off-by: Patrick Georgi <pgeorgi at chromium.org>
    Original-Commit-Id: 1c171f739497fcd26589976676ab94b23cd7ee8b
    Original-Change-Id: I4aa38379ac356114fc91a32cced2fa45a00a09d6
    Original-Signed-off-by: Vadim Bendebury <vbendeb at chromium.org>
    Original-Reviewed-on: https://chromium-review.googlesource.com/262714
    Original-Reviewed-by: Ionela Voinescu <ionela.voinescu at imgtec.com>
    Original-Tested-by: Ionela Voinescu <ionela.voinescu at imgtec.com>
    Original-Reviewed-by: David Hendricks <dhendrix at chromium.org>
    Reviewed-on: http://review.coreboot.org/9891
    Tested-by: build bot (Jenkins)
    Reviewed-by: Stefan Reinauer <stefan.reinauer at coreboot.org>


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

-gerrit



More information about the coreboot-gerrit mailing list