diff options
| author | Patrick Georgi <pgeorgi@chromium.org> | 2016-12-14 16:10:00 +0100 |
|---|---|---|
| committer | Patrick Georgi <pgeorgi@google.com> | 2016-12-15 22:19:17 +0100 |
| commit | 862df924e34927bb05f8d9fde10582789748cb7c (patch) | |
| tree | ff4d263897e31286b61422767ff766a871335a04 | |
| parent | a2ce710df749f06bcdfa95329491046870f3c729 (diff) | |
| download | coreboot-862df924e34927bb05f8d9fde10582789748cb7c.tar.xz | |
util/cbfstool: Fix memory leak
Change-Id: I66cb1c88155ef58610bacfb899e0132e4143c7ac
Signed-off-by: Patrick Georgi <pgeorgi@chromium.org>
Found-by: Coverity Scan #1325836
Reviewed-on: https://review.coreboot.org/17859
Reviewed-by: Nico Huber <nico.h@gmx.de>
Tested-by: build bot (Jenkins)
Reviewed-by: Paul Menzel <paulepanter@users.sourceforge.net>
| -rw-r--r-- | util/cbfstool/cbfs_image.c | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/util/cbfstool/cbfs_image.c b/util/cbfstool/cbfs_image.c index ee229b365d..c34ca8366c 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; |