[coreboot-gerrit] Patch merged into coreboot/master: lib/tpm2_marshaling: fix in correct buffer space semantics

gerrit at coreboot.org gerrit at coreboot.org
Thu Mar 9 19:14:27 CET 2017


the following patch was just integrated into master:
commit 06f12f919f4b7acde88392fd9dce461701ef994e
Author: Aaron Durbin <adurbin at chromium.org>
Date:   Mon Mar 6 16:33:57 2017 -0600

    lib/tpm2_marshaling: fix in correct buffer space semantics
    
    marshal_blob() was setting an unsigned size (size_t) to a value
    of -1 when an error is determined. This is wrong for the current
    implementation of the code because the code assumes the buffer
    space gets set to 0. Setting an unsigned value to -1 effectively
    tells the library the buffer has unlimited amount of space.
    
    BUG=b:35775104
    
    Change-Id: I677a1fd7528bef3ea7420d0a8d0a290e9b15cea3
    Signed-off-by: Aaron Durbin <adurbin at chromium.org>
    Reviewed-on: https://review.coreboot.org/18678
    Reviewed-by: Duncan Laurie <dlaurie at chromium.org>
    Reviewed-by: Furquan Shaikh <furquan at google.com>
    Tested-by: build bot (Jenkins)


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

-gerrit



More information about the coreboot-gerrit mailing list