Jakub Czapiga has submitted this change. ( https://review.coreboot.org/c/coreboot/+/76848?usp=email )
Change subject: payloads/coreinfo: Use C99 flexible arrays ......................................................................
payloads/coreinfo: Use C99 flexible arrays
Use C99 flexible arrays instead of older style of one-element or zero-length arrays. It allows the compiler to generate errors when the flexible array does not occur at the end in the structure.
Change-Id: I17811256b04a17539d3ed77f406892ae77e97515 Signed-off-by: Elyes Haouas ehaouas@noos.fr Reviewed-on: https://review.coreboot.org/c/coreboot/+/76848 Reviewed-by: Felix Singer service+coreboot-gerrit@felixsinger.de Tested-by: build bot (Jenkins) no-reply@coreboot.org --- M payloads/coreinfo/bootlog_module.c M payloads/coreinfo/cbfs_module.c 2 files changed, 3 insertions(+), 3 deletions(-)
Approvals: Felix Singer: Looks good to me, approved build bot (Jenkins): Verified
diff --git a/payloads/coreinfo/bootlog_module.c b/payloads/coreinfo/bootlog_module.c index 5a87f28..07fa814 100644 --- a/payloads/coreinfo/bootlog_module.c +++ b/payloads/coreinfo/bootlog_module.c @@ -17,7 +17,7 @@ struct cbmem_console { u32 size; u32 cursor; - u8 body[0]; + u8 body[]; } __packed;
#define CURSOR_MASK ((1 << 28) - 1) diff --git a/payloads/coreinfo/cbfs_module.c b/payloads/coreinfo/cbfs_module.c index 93c90e7..c8fa7e2 100644 --- a/payloads/coreinfo/cbfs_module.c +++ b/payloads/coreinfo/cbfs_module.c @@ -31,7 +31,7 @@ u32 align; u32 offset; u32 architecture; - u32 pad[1]; + u32 pad[]; } __packed;
struct cbfile { @@ -40,7 +40,7 @@ u32 type; u32 checksum; u32 offset; - char filename[0]; + char filename[]; } __packed;
static int filecount = 0, selected = 0, start_row = 0;