From: Eduardo Habkost <ehabkost@redhat.com>
Date: Thu, 10 Dec 2020 20:47:12 +0000 (-0500)
Subject: csm: Don't check SMBios21Addr before calling copy_smbios_21()
X-Git-Tag: rel-1.16.0~31
X-Git-Url: http://xenbits.xensource.com/gitweb?a=commitdiff_plain;h=107391619a0dafd5900e8933e72dd6fdae97d2d1;p=seabios.git

csm: Don't check SMBios21Addr before calling copy_smbios_21()

copy_smbios_21() already checks if SMBios21Addr is NULL, there's
no need to check it before calling the function.

Signed-off-by: Eduardo Habkost <ehabkost@redhat.com>
---

diff --git a/src/fw/csm.c b/src/fw/csm.c
index 7406902..bc14a92 100644
--- a/src/fw/csm.c
+++ b/src/fw/csm.c
@@ -172,7 +172,7 @@ handle_csm_0002(struct bregs *regs)
 
     // SMBIOS table needs to be copied into the f-seg
     // XX: OVMF doesn't seem to set SmbiosTableLength so don't check it
-    if (csm_boot_table->SmbiosTable && !SMBios21Addr)
+    if (csm_boot_table->SmbiosTable)
         copy_smbios_21((void *)csm_boot_table->SmbiosTable);
 
     // MPTABLE is just there; we don't care where.