On Mon, Aug 26, 2013 at 04:12:50PM -0600, Stephen Warren wrote:
> From: Stephen Warren <[email protected]>
> 
> In real-world use-cases, hashing zero-length data likely never happens.
> However, it is relevant when testing cbootimage with a dummy zero-length
> bootloader binary, e.g.:
> 
> touch u-boot.bin
> cbootimage -t30 ../tamonten-ng/tegra30.img.cfg tegra30-tec-ng.img
> 
> In this scenario, it's useful to create a consistent hash, so that one
> can compare the resultant images before and after applying patches, to
> check for regressions.
> 
> Hence, zero out the hash data so it has consistent content if it isn't
> written to.
> 
> Signed-off-by: Stephen Warren <[email protected]>
> ---
> v2: clear region using calloc at allocation time, rather than memset
>     at hash-calculation time.
> ---
>  src/crypto.c      | 2 +-
>  src/data_layout.c | 2 +-
>  2 files changed, 2 insertions(+), 2 deletions(-)

Looks good:

Reviewed-by: Thierry Reding <[email protected]>

Attachment: pgp4G44p30gnq.pgp
Description: PGP signature

Reply via email to