Elyes Haouas has uploaded this change for review. ( https://review.coreboot.org/c/coreboot/+/84168?usp=email )
Change subject: bsd/metadata_hash.h: Fix Wunterminated-string-initialization error on metadata_hash_anchor ......................................................................
bsd/metadata_hash.h: Fix Wunterminated-string-initialization error on metadata_hash_anchor
while on it, fix "metadata_hash.c" file name in comment.
Change-Id: Ife62d9f7e1e403df130bb4aacacb4897af920460 Signed-off-by: Elyes Haouas ehaouas@noos.fr --- M src/commonlib/bsd/include/commonlib/bsd/metadata_hash.h 1 file changed, 2 insertions(+), 2 deletions(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/68/84168/1
diff --git a/src/commonlib/bsd/include/commonlib/bsd/metadata_hash.h b/src/commonlib/bsd/include/commonlib/bsd/metadata_hash.h index d5e54b5..4f9fa8f 100644 --- a/src/commonlib/bsd/include/commonlib/bsd/metadata_hash.h +++ b/src/commonlib/bsd/include/commonlib/bsd/metadata_hash.h @@ -8,7 +8,7 @@
/* This structure is embedded somewhere in the (uncompressed) bootblock. */ struct metadata_hash_anchor { - uint8_t magic[8]; + uint8_t magic[9]; struct vb2_hash cbfs_hash; /* NOTE: This is just reserving space. sizeof(struct vb2_hash) may change between configurations/versions and cannot be relied upon, so the FMAP hash must be placed @@ -26,7 +26,7 @@ /* * Do not use this constant anywhere else in coreboot code to ensure the bit pattern really only * appears once in the CBFS image. The only coreboot file allowed to use this is - * src/lib/metadata_anchor.c to define the actual anchor data structure. It is defined here so + * src/lib/metadata_hash.c to define the actual anchor data structure. It is defined here so * that it can be shared with cbfstool (which may use it freely). */ #define DO_NOT_USE_METADATA_HASH_ANCHOR_MAGIC_DO_NOT_USE "\xadMdtHsh\x15"