[coreboot-gerrit] Patch merged into coreboot/master: 667c7a3 util/fletcher: fix debug option parsing

gerrit at coreboot.org gerrit at coreboot.org
Sat Oct 18 14:45:58 CEST 2014


the following patch was just integrated into master:
commit 667c7a3b23e154254a4b91286cbb0f6aedf4c410
Author: Patrick Georgi <patrick at georgi-clan.de>
Date:   Mon Sep 29 10:12:29 2014 +0200

    util/fletcher: fix debug option parsing
    
    sizeof(char[]-type+1) isn't very useful. Since one of
    the strings is constant, we also don't need to use
    strncmp that string's length. While at it, str*cmp don't
    return booleans, so check for value instead of faux bools.
    
    Change-Id: Iebb194a60eac454dafeade75f135df92068cf4ab
    Found-by: Coverity Scan
    Signed-off-by: Patrick Georgi <patrick at georgi-clan.de>
    Reviewed-on: http://review.coreboot.org/6988
    Tested-by: build bot (Jenkins)
    Reviewed-by: Edward O'Callaghan <eocallaghan at alterapraxis.com>


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

-gerrit



More information about the coreboot-gerrit mailing list