From a60e7b6da533c119cae0b83207222f7b94f40539 Mon Sep 17 00:00:00 2001 From: Patrick Georgi Date: Tue, 25 Aug 2015 22:26:02 +0200 Subject: cbfstool: use struct cbfs_file * instead of void * My concern was that compilers may something stupid under the assumption of a fixed struct size, but filename is already variable, so things are okay. Change-Id: I5348faf68f0a7993294e9de4c0b6c737278b28af Signed-off-by: Patrick Georgi Reviewed-on: http://review.coreboot.org/11331 Tested-by: build bot (Jenkins) Reviewed-by: Aaron Durbin --- util/cbfstool/cbfs_image.c | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) (limited to 'util/cbfstool/cbfs_image.c') diff --git a/util/cbfstool/cbfs_image.c b/util/cbfstool/cbfs_image.c index 7066b99f31..53ce604764 100644 --- a/util/cbfstool/cbfs_image.c +++ b/util/cbfstool/cbfs_image.c @@ -457,7 +457,7 @@ static int cbfs_add_entry_at(struct cbfs_image *image, struct cbfs_file *entry, const void *data, uint32_t content_offset, - const void *header_data, + const struct cbfs_file *header, uint32_t header_size) { struct cbfs_file *next = cbfs_find_next_entry(image, entry); @@ -487,7 +487,7 @@ static int cbfs_add_entry_at(struct cbfs_image *image, } len = content_offset - addr - header_size; - memcpy(entry, header_data, header_size); + memcpy(entry, header, header_size); if (len != 0) { /* the header moved backwards a bit to accomodate cbfs_file * alignment requirements, so patch up ->offset to still point @@ -533,14 +533,14 @@ static int cbfs_add_entry_at(struct cbfs_image *image, int cbfs_add_entry(struct cbfs_image *image, struct buffer *buffer, uint32_t content_offset, - void *header, uint32_t header_size) + struct cbfs_file *header, uint32_t header_size) { assert(image); assert(buffer); assert(buffer->data); assert(!IS_TOP_ALIGNED_ADDRESS(content_offset)); - const char *name = ((struct cbfs_file *)header)->filename; + const char *name = header->filename; uint32_t entry_type; uint32_t addr, addr_next; -- cgit v1.2.3