[coreboot-gerrit] Patch merged into coreboot/master: cbfstool: Check arguments to strtoul() where appropriate

gerrit at coreboot.org gerrit at coreboot.org
Wed Aug 3 09:42:19 CEST 2016


the following patch was just integrated into master:
commit 9ade7175712a6ebab3a1abd188208b07a2a3fd76
Author: Nico Huber <nico.h at gmx.de>
Date:   Mon Aug 1 21:37:42 2016 +0200

    cbfstool: Check arguments to strtoul() where appropriate
    
    The interface to strtoul() is a weird mess. It may or may not set errno
    if no conversion is done. So check for empty strings and trailing
    characters.
    
    Change-Id: I82373d2a0102fc89144bd12376b5ea3b10c70153
    Signed-off-by: Nico Huber <nico.h at gmx.de>
    Reviewed-on: https://review.coreboot.org/16012
    Tested-by: build bot (Jenkins)
    Reviewed-by: Idwer Vollering <vidwer at gmail.com>
    Reviewed-by: Julius Werner <jwerner at chromium.org>
    Reviewed-by: Paul Menzel <paulepanter at users.sourceforge.net>
    Reviewed-by: Stefan Reinauer <stefan.reinauer at coreboot.org>
    Reviewed-by: Patrick Georgi <pgeorgi at google.com>


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

-gerrit



More information about the coreboot-gerrit mailing list