diff options
| author | Elyes Haouas <ehaouas@noos.fr> | 2023-07-30 13:06:10 +0200 |
|---|---|---|
| committer | Jakub Czapiga <jacz@semihalf.com> | 2023-08-01 12:51:25 +0000 |
| commit | fc2f304f062bc51cf51ad999caf13a6acc64a1b3 (patch) | |
| tree | 4b4da22e0b78bdff9221b3814bc300b3f0fead40 | |
| parent | dc15867e3b938b6349fed3e809931657bdd24da6 (diff) | |
util: 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: I6b87680ec9f501945ae266ae4e4927efd2399d56
Signed-off-by: Elyes Haouas <ehaouas@noos.fr>
Reviewed-on: https://review.coreboot.org/c/coreboot/+/76815
Reviewed-by: Felix Singer <service+coreboot-gerrit@felixsinger.de>
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
| -rw-r--r-- | util/cbfstool/ifwitool.c | 4 | ||||
| -rw-r--r-- | util/cbmem/cbmem.c | 2 | ||||
| -rw-r--r-- | util/intelvbttool/intelvbttool.c | 2 | ||||
| -rw-r--r-- | util/nvramtool/coreboot_tables.h | 6 |
4 files changed, 7 insertions, 7 deletions
diff --git a/util/cbfstool/ifwitool.c b/util/cbfstool/ifwitool.c index efcb594360..453d85c44b 100644 --- a/util/cbfstool/ifwitool.c +++ b/util/cbfstool/ifwitool.c @@ -70,7 +70,7 @@ struct bpdt_entry { struct bpdt { struct bpdt_header h; /* In practice, this could be an array of 0 to n entries. */ - struct bpdt_entry e[0]; + struct bpdt_entry e[]; } __packed; static inline size_t get_bpdt_size(struct bpdt_header *h) @@ -124,7 +124,7 @@ struct subpart_dir_entry { struct subpart_dir { struct subpart_dir_header h; /* In practice, this could be an array of 0 to n entries. */ - struct subpart_dir_entry e[0]; + struct subpart_dir_entry e[]; } __packed; static inline size_t subpart_dir_size(struct subpart_dir_header *h) diff --git a/util/cbmem/cbmem.c b/util/cbmem/cbmem.c index 73c410cbb7..d33b714d22 100644 --- a/util/cbmem/cbmem.c +++ b/util/cbmem/cbmem.c @@ -1116,7 +1116,7 @@ static void dump_tpm_log(void) struct cbmem_console { u32 size; u32 cursor; - u8 body[0]; + u8 body[]; } __attribute__ ((__packed__)); #define CBMC_CURSOR_MASK ((1 << 28) - 1) diff --git a/util/intelvbttool/intelvbttool.c b/util/intelvbttool/intelvbttool.c index c8d7a088d3..a70b33c76b 100644 --- a/util/intelvbttool/intelvbttool.c +++ b/util/intelvbttool/intelvbttool.c @@ -157,7 +157,7 @@ struct bdb_general_definitions { * number = (block_size - sizeof(bdb_general_definitions))/ * sizeof(child_device_config); */ - struct common_child_dev_config devices[0]; + struct common_child_dev_config devices[]; } __attribute__ ((packed)); struct bdb_driver_features { diff --git a/util/nvramtool/coreboot_tables.h b/util/nvramtool/coreboot_tables.h index aafeab4964..d4db00c862 100644 --- a/util/nvramtool/coreboot_tables.h +++ b/util/nvramtool/coreboot_tables.h @@ -90,7 +90,7 @@ struct lb_memory_range { struct lb_memory { uint32_t tag; uint32_t size; - struct lb_memory_range map[0]; + struct lb_memory_range map[]; }; #define LB_TAG_HWRPB 0x0002 @@ -106,7 +106,7 @@ struct lb_mainboard { uint32_t size; uint8_t vendor_idx; uint8_t part_number_idx; - uint8_t strings[0]; + uint8_t strings[]; }; #define LB_TAG_VERSION 0x0004 @@ -122,7 +122,7 @@ struct lb_mainboard { struct lb_string { uint32_t tag; uint32_t size; - uint8_t string[0]; + uint8_t string[]; }; #define LB_TAG_SERIAL 0x000f #define LB_TAG_CONSOLE 0x0010 |