Elyes Haouas has uploaded this change for review.

View Change

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"

To view, visit change 84168. To unsubscribe, or for help writing mail filters, visit settings.