diff options
| author | Bill XIE <persmule@hardenedlinux.org> | 2023-04-07 02:04:21 +0800 |
|---|---|---|
| committer | Lean Sheng Tan <sheng.tan@9elements.com> | 2023-04-16 14:04:32 +0000 |
| commit | cd3545556ee410dc722e7046c96ac0fc7da51ade (patch) | |
| tree | f33c27828f2b85c50abc7657f7898b4cea6575e0 /util/cbfstool | |
| parent | 7f66adbc71e40f890d72ab2882cbb4c6968e0cac (diff) | |
util/cbfstool: Qualify struct e820entry as packed
In order to accord with grub (see include/grub/i386/linux.h) and
comments for offsets of members of struct linux_params,
struct e820entry should be defined as __packed, otherwise,
sizeof(struct linux_params) will become 4224 (0x1080).
Fortunately, the affected area is usually not occupied.
Signed-off-by: Bill XIE <persmule@hardenedlinux.org>
Change-Id: I09955c90e4eec337adca383e628a8821075381d6
Reviewed-on: https://review.coreboot.org/c/coreboot/+/74270
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Reviewed-by: Arthur Heymans <arthur@aheymans.xyz>
Reviewed-by: Paul Menzel <paulepanter@mailbox.org>
Reviewed-by: Nico Huber <nico.h@gmx.de>
Diffstat (limited to 'util/cbfstool')
| -rw-r--r-- | util/cbfstool/linux.h | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/util/cbfstool/linux.h b/util/cbfstool/linux.h index dc4575ca95..2905f35b97 100644 --- a/util/cbfstool/linux.h +++ b/util/cbfstool/linux.h @@ -27,7 +27,7 @@ struct e820entry { #define E820_RESERVED 2 #define E820_ACPI 3 /* usable as RAM once ACPI tables have been read */ #define E820_NVS 4 -}; +} __packed; /* The header of Linux/i386 kernel */ struct linux_header { |