[coreboot-gerrit] Patch set updated for coreboot: util/cbfstool: Handle error condition more carefully

[Patrick Georgi]

Patrick Georgi ([e-mail]) just uploaded a new patch set to gerrit, which you can find at https://review.coreboot.org/17861

-gerrit

commit 19e7e80e1b335741beb6beef8c8b5fccc0ace6ad
Author: Patrick Georgi 
Date:   Wed Dec 14 16:16:47 2016 +0100

util/cbfstool: Handle error condition more carefully

Change-Id: I72a7776d530d1cf0b8fa39e558990df3dc7f7805
    Signed-off-by: Patrick Georgi 
    Found-by: Coverity Scan #1295494
---
 util/cbfstool/common.c | 5 +++--
 1 file changed, 3 insertions(+), 2 deletions(-)

diff --git a/util/cbfstool/common.c b/util/cbfstool/common.c
index 3093ba1..45aec1b 100644
--- a/util/cbfstool/common.c
+++ b/util/cbfstool/common.c
@@ -71,12 +71,13 @@ int buffer_from_file(struct buffer *buffer, const char *filename)
    	return -1;
    }
    buffer->offset = 0;
-	buffer->size = get_file_size(fp);
-	if (buffer->size == -1u) {
+	int file_size = get_file_size(fp);
+	if (file_size < 0) {
    	fprintf(stderr, "could not determine size of %s\n", filename);
    	fclose(fp);
    	return -1;
    }
+	buffer->size = file_size;
    buffer->name = strdup(filename);
    buffer->data = (char *)malloc(buffer->size);
    assert(buffer->data);