Paul Menzel (paulepanter@users.sourceforge.net) just uploaded a new patch set to gerrit, which you can find at http://review.coreboot.org/3770
-gerrit
commit c80921039dc145550e05325cc0be276ffd88ee1b Author: Paul Menzel paulepanter@users.sourceforge.net Date: Sun Jul 14 10:37:10 2013 +0200
util/cbfstool/cbfs_image.c: Store next entry in `next` so difference is not 0
Currently `entry` is overwritten causing the difference to vanish and resulting in a segmentation fault.
Change-Id: Ib563e0ba89f437c5f01c2bf8ed844735a528d2d2 Signed-off-by: Paul Menzel paulepanter@users.sourceforge.net --- util/cbfstool/cbfs_image.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/util/cbfstool/cbfs_image.c b/util/cbfstool/cbfs_image.c index eb9d574..ae4d69f 100644 --- a/util/cbfstool/cbfs_image.c +++ b/util/cbfstool/cbfs_image.c @@ -396,7 +396,7 @@ int cbfs_add_entry(struct cbfs_image *image, struct buffer *buffer,
// setup new entry DEBUG("Setting new empty entry.\n"); - entry = cbfs_find_next_entry(image, entry); + next = cbfs_find_next_entry(image, entry); new_size = (cbfs_get_entry_addr(image, next) - cbfs_get_entry_addr(image, entry)); new_size -= cbfs_calculate_file_header_size("");