summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorMaximilian Brune <maximilian.brune@9elements.com>2023-03-27 04:04:29 +0200
committerFelix Held <felix-coreboot@felixheld.de>2023-04-04 13:22:27 +0000
commit7d6ae2b72c0a4b521d2492fab606e1544f72e979 (patch)
tree977379d4caafd7805e940144a57743925b83ce79
parent991e96083f44f6702876b0f9501bd90c00a4f294 (diff)
arch/x86/smbios: Check str for NULL in smbios_add_string()
Signed-off-by: Maximilian Brune <maximilian.brune@9elements.com> Change-Id: Ic228b869aea362c1f07e0808c2735ff3b285a6bd Reviewed-on: https://review.coreboot.org/c/coreboot/+/73980 Reviewed-by: Eric Lai <eric_lai@quanta.corp-partner.google.com> Tested-by: build bot (Jenkins) <no-reply@coreboot.org> Reviewed-by: Paul Menzel <paulepanter@mailbox.org>
-rw-r--r--src/arch/x86/smbios.c2
1 files changed, 1 insertions, 1 deletions
diff --git a/src/arch/x86/smbios.c b/src/arch/x86/smbios.c
index 188fb4256f..1ec0ad20fe 100644
--- a/src/arch/x86/smbios.c
+++ b/src/arch/x86/smbios.c
@@ -45,7 +45,7 @@ int smbios_add_string(u8 *start, const char *str)
* Return 0 as required for empty strings.
* See Section 6.1.3 "Text Strings" of the SMBIOS specification.
*/
- if (*str == '\0')
+ if (str == NULL || *str == '\0')
return 0;
for (;;) {