Patrick Georgi (pgeorgi@google.com) just uploaded a new patch set to gerrit, which you can find at https://review.coreboot.org/17859
-gerrit
commit 2f4405433ec02a75ae6260a1e9d607c025a0c7fb Author: Patrick Georgi pgeorgi@chromium.org Date: Wed Dec 14 16:10:00 2016 +0100
util/cbfstool: Fix memory leak
Change-Id: I66cb1c88155ef58610bacfb899e0132e4143c7ac Signed-off-by: Patrick Georgi pgeorgi@chromium.org Found-by: Coverity Scan #1325836 --- util/cbfstool/cbfs_image.c | 1 + 1 file changed, 1 insertion(+)
diff --git a/util/cbfstool/cbfs_image.c b/util/cbfstool/cbfs_image.c index ee229b3..c34ca83 100644 --- a/util/cbfstool/cbfs_image.c +++ b/util/cbfstool/cbfs_image.c @@ -1411,6 +1411,7 @@ int cbfs_print_entry_info(struct cbfs_image *image, struct cbfs_file *entry, ntohl(entry->len), hash_type, local_hash, hash_len) != VB2_SUCCESS) { fprintf(fp, "failed to hash '%s'\n", name); + free(hash_str); break; } int valid = memcmp(local_hash, hash->hash_data, hash_len) == 0;