[coreboot-gerrit] Patch merged into coreboot/master: c1a20f0 LZMA: Add a version of ulzma which takes the input and output buffer sizes.

gerrit at coreboot.org gerrit at coreboot.org
Sun Aug 10 22:29:54 CEST 2014


the following patch was just integrated into master:
commit c1a20f0cb8d326d3b6ef5477dc4d7ee9f095665c
Author: Gabe Black <gabeblack at google.com>
Date:   Sat Aug 10 09:35:56 2013 -0700

    LZMA: Add a version of ulzma which takes the input and output buffer sizes.
    
    This new version is used to implement the version which doesn't take the
    input and output buffer sizes.
    
    Old-Change-Id: I8935024aca0849bc939263d7fc3036c586e63c68
    Signed-off-by: Gabe Black <gabeblack at google.com>
    Reviewed-on: https://gerrit.chromium.org/gerrit/65510
    Reviewed-by: Kees Cook <keescook at chromium.org>
    Reviewed-by: Stefan Reinauer <reinauer at google.com>
    Tested-by: Gabe Black <gabeblack at chromium.org>
    Commit-Queue: Gabe Black <gabeblack at chromium.org>
    (cherry picked from commit 465d167ad2f6a67d0b2c91fb6c68c8f9a09dd395)
    
    libpayload: Make lzma truncation non-fatal.
    
    If the size the lzma header claims it needs is bigger than the space we have,
    print a message and continue rather than erroring out. Apparently the encoder
    is lazy sometimes and just puts a large value there regardless of what the
    actual size is.
    
    This was the original intention for this code, but an outdated version of the
    patch ended up being submitted.
    
    Old-Change-Id: Ibcf7ac0fd4b65ce85377421a4ee67b82d92d29d3
    Signed-off-by: Gabe Black <gabeblack at google.com>
    Reviewed-on: https://gerrit.chromium.org/gerrit/66235
    Reviewed-by: Stefan Reinauer <reinauer at google.com>
    Commit-Queue: Gabe Black <gabeblack at chromium.org>
    Tested-by: Gabe Black <gabeblack at chromium.org>
    (cherry picked from commit 30c628eeada274fc8b94f8f69f9df4f33cbfc773)
    
    Squashed two related commits and updated the commit message to be
    more clear.
    
    Change-Id: I484b5c1e3809781033d146609a35a9e5e666c8ed
    Signed-off-by: Isaac Christensen <isaac.christensen at se-eng.com>
    Reviewed-on: http://review.coreboot.org/6408
    Tested-by: build bot (Jenkins)
    Reviewed-by: Patrick Georgi <patrick at georgi-clan.de>


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

-gerrit



More information about the coreboot-gerrit mailing list