From 48169e80364b541b81620892a7cb51ad46ceb04a Mon Sep 17 00:00:00 2001 From: Maximilian Brune Date: Wed, 2 Aug 2023 13:20:00 +0200 Subject: lib/smbios: Add 32 bit entry point if below 4G If the smbios table is not below 4G there is no need to have a 32 bit entry point. Even worse it could cause the payload to try to use the entry point. Signed-off-by: Maximilian Brune Change-Id: I4cb426bb0c45282ed03ff4c65d15004b7f985dab Reviewed-on: https://review.coreboot.org/c/coreboot/+/76911 Tested-by: build bot (Jenkins) Reviewed-by: Arthur Heymans Reviewed-by: Lean Sheng Tan --- src/lib/smbios.c | 43 ++++++++++++++++++++++++------------------- 1 file changed, 24 insertions(+), 19 deletions(-) (limited to 'src/lib') diff --git a/src/lib/smbios.c b/src/lib/smbios.c index 51f1593075..c6cfb05869 100644 --- a/src/lib/smbios.c +++ b/src/lib/smbios.c @@ -1203,7 +1203,7 @@ static int smbios_walk_device_tree(struct device *tree, int *handle, unsigned lo unsigned long smbios_write_tables(unsigned long current) { - struct smbios_entry *se; + struct smbios_entry *se = NULL; struct smbios_entry30 *se3; unsigned long tables; int len = 0; @@ -1213,9 +1213,12 @@ unsigned long smbios_write_tables(unsigned long current) current = ALIGN_UP(current, 16); printk(BIOS_DEBUG, "%s: %08lx\n", __func__, current); - se = (struct smbios_entry *)current; - current += sizeof(*se); - current = ALIGN_UP(current, 16); + // only add a 32 bit entry point if SMBIOS table is below 4G + if (current < UINT32_MAX) { + se = (struct smbios_entry *)current; + current += sizeof(*se); + current = ALIGN_UP(current, 16); + } se3 = (struct smbios_entry30 *)current; current += sizeof(*se3); @@ -1253,21 +1256,23 @@ unsigned long smbios_write_tables(unsigned long current) update_max(len, max_struct_size, smbios_write_type127(¤t, handle++)); - /* Install SMBIOS 2.1 entry point */ - memset(se, 0, sizeof(*se)); - memcpy(se->anchor, "_SM_", 4); - se->length = sizeof(*se); - se->major_version = 3; - se->minor_version = 0; - se->max_struct_size = max_struct_size; - se->struct_count = handle; - memcpy(se->intermediate_anchor_string, "_DMI_", 5); - - se->struct_table_address = (u32)tables; - se->struct_table_length = len; - - se->intermediate_checksum = smbios_checksum((u8 *)se + 0x10, sizeof(*se) - 0x10); - se->checksum = smbios_checksum((u8 *)se, sizeof(*se)); + if (se) { + /* Install SMBIOS 2.1 entry point */ + memset(se, 0, sizeof(*se)); + memcpy(se->anchor, "_SM_", 4); + se->length = sizeof(*se); + se->major_version = 3; + se->minor_version = 0; + se->max_struct_size = max_struct_size; + se->struct_count = handle; + memcpy(se->intermediate_anchor_string, "_DMI_", 5); + + se->struct_table_address = (u32)tables; + se->struct_table_length = len; + + se->intermediate_checksum = smbios_checksum((u8 *)se + 0x10, sizeof(*se) - 0x10); + se->checksum = smbios_checksum((u8 *)se, sizeof(*se)); + } /* Install SMBIOS 3.0 entry point */ memset(se3, 0, sizeof(*se3)); -- cgit v1.2.3