commit dc9dbc0059af620efa919ac3e0d3b92abd50e658
author Patrick Georgi <patrick@georgi-clan.de> Tue Aug 25 20:17:14 2015 +0200
committer Patrick Georgi <pgeorgi@google.com> Thu Aug 27 15:01:22 2015 +0000
tree 04d268e80a1f4a264a61a886893ee845fb8bf91d
parent e590358064a1db077a3dafec473433260a3ba646

cbfstool: passing a header is now mandatory for converters

Change-Id: Iea5377af735b06bcaefb999547a8896b1c70763a
Signed-off-by: Patrick Georgi <patrick@georgi-clan.de>
Reviewed-on: http://review.coreboot.org/11330
Tested-by: build bot (Jenkins)
Reviewed-by: Aaron Durbin <adurbin@chromium.org>

util/cbfstool/cbfstool.c

diff --git a/util/cbfstool/cbfstool.c b/util/cbfstool/cbfstool.c
index 465580c..0939a06 100644
--- a/util/cbfstool/cbfstool.c
+++ b/util/cbfstool/cbfstool.c
@@ -247,8 +247,7 @@
 	buffer_delete(buffer);
 	// direct assign, no dupe.
 	memcpy(buffer, &output, sizeof(*buffer));
-	if (header)
-		header->len = htonl(output.size);
+	header->len = htonl(output.size);
 	return 0;
 }
 
@@ -279,8 +278,7 @@
 	buffer_delete(buffer);
 	// direct assign, no dupe.
 	memcpy(buffer, &output, sizeof(*buffer));
-	if (header)
-		header->len = htonl(output.size);
+	header->len = htonl(output.size);
 	return 0;
 }
 
@@ -297,8 +295,7 @@
 	buffer_delete(buffer);
 	// direct assign, no dupe.
 	memcpy(buffer, &output, sizeof(*buffer));
-	if (header)
-		header->len = htonl(output.size);
+	header->len = htonl(output.size);
 	return 0;
 }