[coreboot-gerrit] New patch to review for coreboot: cbfstool: factor out creating a cbfs file header

[Patrick Georgi]

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

-gerrit

commit 936c8dcb4e3d3f95f783a90ccf497fe3153e5211
Author: Patrick Georgi 
Date:   Wed Aug 12 09:20:11 2015 +0200

cbfstool: factor out creating a cbfs file header

We will want to create headers that live outside the final image at some point
    (eg. to build the file before we even know where to place it).

Change-Id: Ie4c0323df8d5be955aec3621b75309e8f11fae49
    Signed-off-by: Patrick Georgi 
---
 util/cbfstool/cbfs_image.c | 16 ++++++++++++++--
 util/cbfstool/cbfs_image.h |  4 ++++
 2 files changed, 18 insertions(+), 2 deletions(-)

diff --git a/util/cbfstool/cbfs_image.c b/util/cbfstool/cbfs_image.c
index b0c3080..077f20f 100644
--- a/util/cbfstool/cbfs_image.c
+++ b/util/cbfstool/cbfs_image.c
@@ -1020,10 +1020,14 @@ int cbfs_is_valid_entry(struct cbfs_image *image, struct cbfs_file *entry)
    					strlen(CBFS_FILE_MAGIC));
 }

-int cbfs_create_empty_entry(struct cbfs_file *entry, int type,
+struct cbfs_file *cbfs_create_file_header(int type,
    		    size_t len, const char *name)
 {
-	memset(entry, CBFS_CONTENT_DEFAULT_VALUE, sizeof(*entry));
+	// assume that there won't be file names of ~1000 bytes
+	const int bufsize = 1024;
+
+	struct cbfs_file *entry = malloc(bufsize);
+	memset(entry, CBFS_CONTENT_DEFAULT_VALUE, bufsize);
    memcpy(entry->magic, CBFS_FILE_MAGIC, sizeof(entry->magic));
    entry->type = htonl(type);
    entry->len = htonl(len);
@@ -1031,6 +1035,14 @@ int cbfs_create_empty_entry(struct cbfs_file *entry, int type,
    entry->offset = htonl(cbfs_calculate_file_header_size(name));
    memset(entry->filename, 0, ntohl(entry->offset) - sizeof(*entry));
    strcpy(entry->filename, name);
+	return entry;
+}
+
+int cbfs_create_empty_entry(struct cbfs_file *entry, int type,
+			    size_t len, const char *name)
+{
+	struct cbfs_file *tmp = cbfs_create_file_header(type, len, name);
+	memcpy(entry, tmp, ntohl(tmp->offset));
    memset(CBFS_SUBHEADER(entry), CBFS_CONTENT_DEFAULT_VALUE, len);
    return 0;
 }
diff --git a/util/cbfstool/cbfs_image.h b/util/cbfstool/cbfs_image.h
index ad999e3..5b27234 100644
--- a/util/cbfstool/cbfs_image.h
+++ b/util/cbfstool/cbfs_image.h
@@ -99,6 +99,10 @@ int cbfs_add_entry(struct cbfs_image *image, struct buffer *buffer,
 /* Removes an entry from CBFS image. Returns 0 on success, otherwise non-zero. */
 int cbfs_remove_entry(struct cbfs_image *image, const char *name);

+/* Create a new cbfs file header structure to work with */
+struct cbfs_file *cbfs_create_file_header(int type, size_t len,
+	const char *name);
+
 /* Initializes a new empty (type = NULL) entry with size and name in CBFS image.
  * Returns 0 on success, otherwise (ex, not found) non-zero. */
 int cbfs_create_empty_entry(struct cbfs_file *entry, int type,