According to PI errata 0000654 and 000811, we need use 0xFFFE to instead of 0 for EFI_SMBIOS_PROTOCOL.Add() SmbiosHandle parameter to assign a unique handle to the SMBIOS record, and for EFI_SMBIOS_PROTOCOL.GetNext() SmbiosHandle parameter to get the first matched SMBIOS handle or indicate no more SMBIOS record.

Signed-off-by: lzeng14
Reviewed-by: li-elvin

git-svn-id: https://edk2.svn.sourceforge.net/svnroot/edk2/trunk/edk2@12753 6f19259b-4bc3-4df7-8a09-765794883524

15 files changed, 87 insertions, 110 deletions

diff --git a/UnixPkg/CpuRuntimeDxe/Cpu.c b/UnixPkg/CpuRuntimeDxe/Cpu.c
index d4926c370..54c784ea6 100644
--- a/UnixPkg/CpuRuntimeDxe/Cpu.c
+++ b/UnixPkg/CpuRuntimeDxe/Cpu.c
@@ -1,6 +1,6 @@
 /*++

 

-Copyright (c) 2006 - 2009, Intel Corporation. All rights reserved.<BR>

+Copyright (c) 2006 - 2011, Intel Corporation. All rights reserved.<BR>

 This program and the accompanying materials

 are licensed and made available under the terms and conditions of the BSD License

 which accompanies this distribution.  The full text of the license may be found at

@@ -417,7 +417,7 @@ LogSmbiosData (
   EFI_STATUS         Status;

   EFI_SMBIOS_HANDLE  SmbiosHandle;

   

-  SmbiosHandle = 0;

+  SmbiosHandle = SMBIOS_HANDLE_PI_RESERVED;

   Status = Smbios->Add (

                      Smbios,

                      NULL,

diff --git a/UnixPkg/MiscSubClassPlatformDxe/MiscBaseBoardManufacturerFunction.c b/UnixPkg/MiscSubClassPlatformDxe/MiscBaseBoardManufacturerFunction.c
index 04f5a1edc..0158ae9d5 100644
--- a/UnixPkg/MiscSubClassPlatformDxe/MiscBaseBoardManufacturerFunction.c
+++ b/UnixPkg/MiscSubClassPlatformDxe/MiscBaseBoardManufacturerFunction.c
@@ -2,7 +2,7 @@
   BaseBoard manufacturer information boot time changes.

   SMBIOS type 2.

 

-  Copyright (c) 2009, Intel Corporation. All rights reserved.<BR>

+  Copyright (c) 2009 - 2011, Intel Corporation. All rights reserved.<BR>

   This program and the accompanying materials

   are licensed and made available under the terms and conditions of the BSD License

   which accompanies this distribution.  The full text of the license may be found at

@@ -154,13 +154,7 @@ MISC_SMBIOS_TABLE_FUNCTION(MiscBaseBoardManufacturer)
   //

   // Now we have got the full smbios record, call smbios protocol to add this record.

   //

-  SmbiosHandle = 0;

-  Status = Smbios-> Add(

-                      Smbios, 

-                      NULL,

-                      &SmbiosHandle, 

-                      (EFI_SMBIOS_TABLE_HEADER *) SmbiosRecord

-                      );

+  Status = AddSmbiosRecord (Smbios, &SmbiosHandle, (EFI_SMBIOS_TABLE_HEADER *) SmbiosRecord);

 

   FreePool(SmbiosRecord);

   return Status;

diff --git a/UnixPkg/MiscSubClassPlatformDxe/MiscBiosVendorFunction.c b/UnixPkg/MiscSubClassPlatformDxe/MiscBiosVendorFunction.c
index 6d64ef463..fbfb52fce 100644
--- a/UnixPkg/MiscSubClassPlatformDxe/MiscBiosVendorFunction.c
+++ b/UnixPkg/MiscSubClassPlatformDxe/MiscBiosVendorFunction.c
@@ -3,7 +3,7 @@
   Misc. subclass type 2.

   SMBIOS type 0.

 

-  Copyright (c) 2009, Intel Corporation. All rights reserved.<BR>

+  Copyright (c) 2009 - 2011, Intel Corporation. All rights reserved.<BR>

   This program and the accompanying materials

   are licensed and made available under the terms and conditions of the BSD License

   which accompanies this distribution.  The full text of the license may be found at

@@ -189,13 +189,7 @@ MISC_SMBIOS_TABLE_FUNCTION(MiscBiosVendor)
   //

   // Now we have got the full smbios record, call smbios protocol to add this record.

   //

-  SmbiosHandle = 0;

-  Status = Smbios-> Add(

-                      Smbios, 

-                      NULL,

-                      &SmbiosHandle, 

-                      (EFI_SMBIOS_TABLE_HEADER *) SmbiosRecord

-                      );

+  Status = AddSmbiosRecord (Smbios, &SmbiosHandle, (EFI_SMBIOS_TABLE_HEADER *) SmbiosRecord);

 

   FreePool(SmbiosRecord);

   return Status;

diff --git a/UnixPkg/MiscSubClassPlatformDxe/MiscBootInformationFunction.c b/UnixPkg/MiscSubClassPlatformDxe/MiscBootInformationFunction.c
index 65c9792a9..fec5ecc0b 100644
--- a/UnixPkg/MiscSubClassPlatformDxe/MiscBootInformationFunction.c
+++ b/UnixPkg/MiscSubClassPlatformDxe/MiscBootInformationFunction.c
@@ -2,7 +2,7 @@
   boot information boot time changes.

   SMBIOS type 32.

 

-  Copyright (c) 2009, Intel Corporation. All rights reserved.<BR>

+  Copyright (c) 2009 - 2011, Intel Corporation. All rights reserved.<BR>

   This program and the accompanying materials

   are licensed and made available under the terms and conditions of the BSD License

   which accompanies this distribution.  The full text of the license may be found at

@@ -61,13 +61,8 @@ MISC_SMBIOS_TABLE_FUNCTION(BootInformationStatus)
   //

   // Now we have got the full smbios record, call smbios protocol to add this record.

   //

-  SmbiosHandle = 0;

-  Status = Smbios-> Add(

-                      Smbios, 

-                      NULL,

-                      &SmbiosHandle, 

-                      (EFI_SMBIOS_TABLE_HEADER *) SmbiosRecord

-                      );

+  Status = AddSmbiosRecord (Smbios, &SmbiosHandle, (EFI_SMBIOS_TABLE_HEADER *) SmbiosRecord);

+

   FreePool(SmbiosRecord);

   return Status;

 }

diff --git a/UnixPkg/MiscSubClassPlatformDxe/MiscChassisManufacturerFunction.c b/UnixPkg/MiscSubClassPlatformDxe/MiscChassisManufacturerFunction.c
index fd1262b19..ce8960347 100644
--- a/UnixPkg/MiscSubClassPlatformDxe/MiscChassisManufacturerFunction.c
+++ b/UnixPkg/MiscSubClassPlatformDxe/MiscChassisManufacturerFunction.c
@@ -2,7 +2,7 @@
   Chassis manufacturer information boot time changes.

   SMBIOS type 3.

 

-  Copyright (c) 2009, Intel Corporation. All rights reserved.<BR>

+  Copyright (c) 2009 - 2011, Intel Corporation. All rights reserved.<BR>

   This program and the accompanying materials

   are licensed and made available under the terms and conditions of the BSD License

   which accompanies this distribution.  The full text of the license may be found at

@@ -124,13 +124,7 @@ MISC_SMBIOS_TABLE_FUNCTION(MiscChassisManufacturer)
   //

   // Now we have got the full smbios record, call smbios protocol to add this record.

   //

-  SmbiosHandle = 0;

-  Status = Smbios-> Add(

-                      Smbios, 

-                      NULL,

-                      &SmbiosHandle, 

-                      (EFI_SMBIOS_TABLE_HEADER *) SmbiosRecord

-                      );

+  Status = AddSmbiosRecord (Smbios, &SmbiosHandle, (EFI_SMBIOS_TABLE_HEADER *) SmbiosRecord);

 

   FreePool(SmbiosRecord);

   return Status;

diff --git a/UnixPkg/MiscSubClassPlatformDxe/MiscNumberOfInstallableLanguagesFunction.c b/UnixPkg/MiscSubClassPlatformDxe/MiscNumberOfInstallableLanguagesFunction.c
index 288848d0d..64addad4a 100644
--- a/UnixPkg/MiscSubClassPlatformDxe/MiscNumberOfInstallableLanguagesFunction.c
+++ b/UnixPkg/MiscSubClassPlatformDxe/MiscNumberOfInstallableLanguagesFunction.c
@@ -2,7 +2,7 @@
   This driver parses the mSmbiosMiscDataTable structure and reports

   any generated data.

 

-  Copyright (c) 2009, Intel Corporation. All rights reserved.<BR>

+  Copyright (c) 2009 - 2011, Intel Corporation. All rights reserved.<BR>

   This program and the accompanying materials

   are licensed and made available under the terms and conditions of the BSD License

   which accompanies this distribution.  The full text of the license may be found at

@@ -226,13 +226,8 @@ MISC_SMBIOS_TABLE_FUNCTION(NumberOfInstallableLanguages)
   //

   // Now we have got the full smbios record, call smbios protocol to add this record.

   //

-  SmbiosHandle = 0;

-  Status = Smbios-> Add(

-                      Smbios, 

-                      NULL,

-                      &SmbiosHandle, 

-                      (EFI_SMBIOS_TABLE_HEADER *) SmbiosRecord

-                      );

+  Status = AddSmbiosRecord (Smbios, &SmbiosHandle, (EFI_SMBIOS_TABLE_HEADER *) SmbiosRecord);

+

   FreePool(SmbiosRecord);

   return Status;

 }

diff --git a/UnixPkg/MiscSubClassPlatformDxe/MiscOemStringFunction.c b/UnixPkg/MiscSubClassPlatformDxe/MiscOemStringFunction.c
index 94fc71c23..f0401cebb 100644
--- a/UnixPkg/MiscSubClassPlatformDxe/MiscOemStringFunction.c
+++ b/UnixPkg/MiscSubClassPlatformDxe/MiscOemStringFunction.c
@@ -2,7 +2,7 @@
   boot information boot time changes.

   SMBIOS type 11.

 

-  Copyright (c) 2009, Intel Corporation. All rights reserved.<BR>

+  Copyright (c) 2009 - 2011, Intel Corporation. All rights reserved.<BR>

   This program and the accompanying materials

   are licensed and made available under the terms and conditions of the BSD License

   which accompanies this distribution.  The full text of the license may be found at

@@ -68,13 +68,8 @@ MISC_SMBIOS_TABLE_FUNCTION(OemString)
   //

   // Now we have got the full smbios record, call smbios protocol to add this record.

   //

-  SmbiosHandle = 0;

-  Status = Smbios-> Add(

-                      Smbios, 

-                      NULL,

-                      &SmbiosHandle, 

-                      (EFI_SMBIOS_TABLE_HEADER *) SmbiosRecord

-                      );

+  Status = AddSmbiosRecord (Smbios, &SmbiosHandle, (EFI_SMBIOS_TABLE_HEADER *) SmbiosRecord);

+

   FreePool(SmbiosRecord);

   return Status;

 }

diff --git a/UnixPkg/MiscSubClassPlatformDxe/MiscPortInternalConnectorDesignatorFunction.c b/UnixPkg/MiscSubClassPlatformDxe/MiscPortInternalConnectorDesignatorFunction.c
index 8f240a408..9f7708be7 100644
--- a/UnixPkg/MiscSubClassPlatformDxe/MiscPortInternalConnectorDesignatorFunction.c
+++ b/UnixPkg/MiscSubClassPlatformDxe/MiscPortInternalConnectorDesignatorFunction.c
@@ -1,6 +1,6 @@
 /*++

  

-Copyright (c) 2006 - 2009, Intel Corporation. All rights reserved.<BR>

+Copyright (c) 2006 - 2011, Intel Corporation. All rights reserved.<BR>

 This program and the accompanying materials                          

 are licensed and made available under the terms and conditions of the BSD License         

 which accompanies this distribution.  The full text of the license may be found at        

@@ -161,13 +161,8 @@ Returns:
   //

   // Now we have got the full smbios record, call smbios protocol to add this record.

   //

-  SmbiosHandle = 0;

-  Status = Smbios-> Add(

-                      Smbios, 

-                      NULL,

-                      &SmbiosHandle, 

-                      (EFI_SMBIOS_TABLE_HEADER *) SmbiosRecord

-                      );

+  Status = AddSmbiosRecord (Smbios, &SmbiosHandle, (EFI_SMBIOS_TABLE_HEADER *) SmbiosRecord);

+

   FreePool(SmbiosRecord);

   return Status;

 }

diff --git a/UnixPkg/MiscSubClassPlatformDxe/MiscResetCapabilitiesFunction.c b/UnixPkg/MiscSubClassPlatformDxe/MiscResetCapabilitiesFunction.c
index 8303c5a29..588f9d65d 100644
--- a/UnixPkg/MiscSubClassPlatformDxe/MiscResetCapabilitiesFunction.c
+++ b/UnixPkg/MiscSubClassPlatformDxe/MiscResetCapabilitiesFunction.c
@@ -2,7 +2,7 @@
   ResetCapabilities.

   SMBIOS type 23.

 

-  Copyright (c) 2009, Intel Corporation. All rights reserved.<BR>

+  Copyright (c) 2009 - 2011, Intel Corporation. All rights reserved.<BR>

   This program and the accompanying materials

   are licensed and made available under the terms and conditions of the BSD License

   which accompanies this distribution.  The full text of the license may be found at

@@ -63,13 +63,8 @@ MISC_SMBIOS_TABLE_FUNCTION(MiscResetCapabilities)
   //

   // Now we have got the full smbios record, call smbios protocol to add this record.

   //

-  SmbiosHandle = 0;

-  Status = Smbios-> Add(

-                      Smbios, 

-                      NULL,

-                      &SmbiosHandle, 

-                      (EFI_SMBIOS_TABLE_HEADER *) SmbiosRecord

-                      );

+  Status = AddSmbiosRecord (Smbios, &SmbiosHandle, (EFI_SMBIOS_TABLE_HEADER *) SmbiosRecord);

+

   FreePool(SmbiosRecord);

   return Status;

 }

diff --git a/UnixPkg/MiscSubClassPlatformDxe/MiscSubClassDriver.h b/UnixPkg/MiscSubClassPlatformDxe/MiscSubClassDriver.h
index debd08bc6..b682aebb0 100644
--- a/UnixPkg/MiscSubClassPlatformDxe/MiscSubClassDriver.h
+++ b/UnixPkg/MiscSubClassPlatformDxe/MiscSubClassDriver.h
@@ -1,6 +1,6 @@
 /*++

 

-Copyright (c) 2006 - 2009, Intel Corporation. All rights reserved.<BR>

+Copyright (c) 2006 - 2011, Intel Corporation. All rights reserved.<BR>

 This program and the accompanying materials

 are licensed and made available under the terms and conditions of the BSD License

 which accompanies this distribution.  The full text of the license may be found at

@@ -105,17 +105,27 @@ extern UINTN                        mMiscSubclassDataTableEntries;
 extern UINT8                        MiscSubclassStrings[];

 extern EFI_HII_HANDLE               mHiiHandle;

 

-//

-// Prototypes

-//

+/**

+  Add an SMBIOS record.

+

+  @param  Smbios                The EFI_SMBIOS_PROTOCOL instance.

+  @param  SmbiosHandle          A unique handle will be assigned to the SMBIOS record.

+  @param  Record                The data for the fixed portion of the SMBIOS record. The format of the record is

+                                determined by EFI_SMBIOS_TABLE_HEADER.Type. The size of the formatted area is defined 

+                                by EFI_SMBIOS_TABLE_HEADER.Length and either followed by a double-null (0x0000) or 

+                                a set of null terminated strings and a null.

+

+  @retval EFI_SUCCESS           Record was added.

+  @retval EFI_OUT_OF_RESOURCES  Record was not added due to lack of system resources.

+

+**/

 EFI_STATUS

-EFIAPI

-MiscSubclassDriverEntryPoint (

-  IN EFI_HANDLE         ImageHandle,

-  IN EFI_SYSTEM_TABLE   *SystemTable

+AddSmbiosRecord (

+  IN EFI_SMBIOS_PROTOCOL        *Smbios,

+  OUT EFI_SMBIOS_HANDLE         *SmbiosHandle,

+  IN EFI_SMBIOS_TABLE_HEADER    *Record

   );

 

-

 #endif /* _MISC_SUBCLASS_DRIVER_H */

 

 /* eof - MiscSubclassDriver.h */

diff --git a/UnixPkg/MiscSubClassPlatformDxe/MiscSubclassDriverEntryPoint.c b/UnixPkg/MiscSubClassPlatformDxe/MiscSubclassDriverEntryPoint.c
index a2c14e7a7..41b32fe67 100644
--- a/UnixPkg/MiscSubClassPlatformDxe/MiscSubclassDriverEntryPoint.c
+++ b/UnixPkg/MiscSubClassPlatformDxe/MiscSubclassDriverEntryPoint.c
@@ -1,6 +1,6 @@
 /*++

 

-Copyright (c) 2006 - 2009, Intel Corporation. All rights reserved.<BR>

+Copyright (c) 2006 - 2011, Intel Corporation. All rights reserved.<BR>

 This program and the accompanying materials

 are licensed and made available under the terms and conditions of the BSD License

 which accompanies this distribution.  The full text of the license may be found at

@@ -90,8 +90,8 @@ LogMemorySmbiosRecord (
   //

   // Generate Memory Array Mapped Address info (TYPE 19)

   //

-  MemArrayMappedAddrSmbiosHandle = 0;

-  Status = Smbios->Add (Smbios, NULL, &MemArrayMappedAddrSmbiosHandle, (EFI_SMBIOS_TABLE_HEADER*) Type19Record);

+  Status = AddSmbiosRecord (Smbios, &MemArrayMappedAddrSmbiosHandle, (EFI_SMBIOS_TABLE_HEADER*) Type19Record);

+

   FreePool(Type19Record);

   ASSERT_EFI_ERROR (Status);

 

@@ -168,3 +168,34 @@ Returns:
   EfiStatus = LogMemorySmbiosRecord();

   return EfiStatus;

 }

+

+/**

+  Add an SMBIOS record.

+

+  @param  Smbios                The EFI_SMBIOS_PROTOCOL instance.

+  @param  SmbiosHandle          A unique handle will be assigned to the SMBIOS record.

+  @param  Record                The data for the fixed portion of the SMBIOS record. The format of the record is

+                                determined by EFI_SMBIOS_TABLE_HEADER.Type. The size of the formatted area is defined 

+                                by EFI_SMBIOS_TABLE_HEADER.Length and either followed by a double-null (0x0000) or 

+                                a set of null terminated strings and a null.

+

+  @retval EFI_SUCCESS           Record was added.

+  @retval EFI_OUT_OF_RESOURCES  Record was not added due to lack of system resources.

+

+**/

+EFI_STATUS

+AddSmbiosRecord (

+  IN EFI_SMBIOS_PROTOCOL        *Smbios,

+  OUT EFI_SMBIOS_HANDLE         *SmbiosHandle,

+  IN EFI_SMBIOS_TABLE_HEADER    *Record

+  )

+{

+  *SmbiosHandle = SMBIOS_HANDLE_PI_RESERVED;

+  return Smbios->Add (

+                   Smbios,

+                   NULL,

+                   SmbiosHandle,

+                   Record

+                   );

+}

+

diff --git a/UnixPkg/MiscSubClassPlatformDxe/MiscSystemLanguageStringFunction.c b/UnixPkg/MiscSubClassPlatformDxe/MiscSystemLanguageStringFunction.c
index 3e10c63ca..86d63b419 100644
--- a/UnixPkg/MiscSubClassPlatformDxe/MiscSystemLanguageStringFunction.c
+++ b/UnixPkg/MiscSubClassPlatformDxe/MiscSystemLanguageStringFunction.c
@@ -2,7 +2,7 @@
   ResetCapabilities.

   SMBIOS type 23.

 

-  Copyright (c) 2009, Intel Corporation. All rights reserved.<BR>

+  Copyright (c) 2009 - 2011, Intel Corporation. All rights reserved.<BR>

   This program and the accompanying materials

   are licensed and made available under the terms and conditions of the BSD License

   which accompanies this distribution.  The full text of the license may be found at

@@ -72,13 +72,8 @@ MISC_SMBIOS_TABLE_FUNCTION(SystemLanguageString)
   //

   // Now we have got the full smbios record, call smbios protocol to add this record.

   //

-  SmbiosHandle = 0;

-  Status = Smbios-> Add(

-                      Smbios, 

-                      NULL,

-                      &SmbiosHandle, 

-                      (EFI_SMBIOS_TABLE_HEADER *) SmbiosRecord

-                      );

+  Status = AddSmbiosRecord (Smbios, &SmbiosHandle, (EFI_SMBIOS_TABLE_HEADER *) SmbiosRecord);

+

   FreePool(SmbiosRecord);

   return Status;

 }

diff --git a/UnixPkg/MiscSubClassPlatformDxe/MiscSystemManufacturerFunction.c b/UnixPkg/MiscSubClassPlatformDxe/MiscSystemManufacturerFunction.c
index 1483ace35..3582447a4 100644
--- a/UnixPkg/MiscSubClassPlatformDxe/MiscSystemManufacturerFunction.c
+++ b/UnixPkg/MiscSubClassPlatformDxe/MiscSystemManufacturerFunction.c
@@ -1,6 +1,6 @@
 /*++

 

-Copyright (c) 2006 - 2009, Intel Corporation. All rights reserved.<BR>

+Copyright (c) 2006 - 2011, Intel Corporation. All rights reserved.<BR>

 This program and the accompanying materials                          

 are licensed and made available under the terms and conditions of the BSD License         

 which accompanies this distribution.  The full text of the license may be found at        

@@ -126,13 +126,8 @@ MISC_SMBIOS_TABLE_FUNCTION(MiscSystemManufacturer)
   //

   // Now we have got the full smbios record, call smbios protocol to add this record.

   //

-  SmbiosHandle = 0;

-  Status = Smbios-> Add(

-                      Smbios, 

-                      NULL,

-                      &SmbiosHandle, 

-                      (EFI_SMBIOS_TABLE_HEADER *) SmbiosRecord

-                      );

+  Status = AddSmbiosRecord (Smbios, &SmbiosHandle, (EFI_SMBIOS_TABLE_HEADER *) SmbiosRecord);

+

   FreePool(SmbiosRecord);

   return Status;

 }

diff --git a/UnixPkg/MiscSubClassPlatformDxe/MiscSystemOptionStringFunction.c b/UnixPkg/MiscSubClassPlatformDxe/MiscSystemOptionStringFunction.c
index a910634f0..145603a9a 100644
--- a/UnixPkg/MiscSubClassPlatformDxe/MiscSystemOptionStringFunction.c
+++ b/UnixPkg/MiscSubClassPlatformDxe/MiscSystemOptionStringFunction.c
@@ -2,7 +2,7 @@
   BIOS system option string boot time changes.

   SMBIOS type 12.

 

-  Copyright (c) 2009, Intel Corporation. All rights reserved.<BR>

+  Copyright (c) 2009 - 2011, Intel Corporation. All rights reserved.<BR>

   This program and the accompanying materials

   are licensed and made available under the terms and conditions of the BSD License

   which accompanies this distribution.  The full text of the license may be found at

@@ -70,13 +70,7 @@ MISC_SMBIOS_TABLE_FUNCTION(SystemOptionString)
   //

   // Now we have got the full smbios record, call smbios protocol to add this record.

   //

-  SmbiosHandle = 0;

-  Status = Smbios-> Add(

-                      Smbios, 

-                      NULL,

-                      &SmbiosHandle, 

-                      (EFI_SMBIOS_TABLE_HEADER *) SmbiosRecord

-                      );

+  Status = AddSmbiosRecord (Smbios, &SmbiosHandle, (EFI_SMBIOS_TABLE_HEADER *) SmbiosRecord);

 

   FreePool(SmbiosRecord);

   return Status;

diff --git a/UnixPkg/MiscSubClassPlatformDxe/MiscSystemSlotDesignationFunction.c b/UnixPkg/MiscSubClassPlatformDxe/MiscSystemSlotDesignationFunction.c
index 72c4137c6..2bf884423 100644
--- a/UnixPkg/MiscSubClassPlatformDxe/MiscSystemSlotDesignationFunction.c
+++ b/UnixPkg/MiscSubClassPlatformDxe/MiscSystemSlotDesignationFunction.c
@@ -2,7 +2,7 @@
   BIOS system slot designator information boot time changes.

   SMBIOS type 9.

 

-  Copyright (c) 2009, Intel Corporation. All rights reserved.<BR>

+  Copyright (c) 2009 - 2011, Intel Corporation. All rights reserved.<BR>

   This program and the accompanying materials

   are licensed and made available under the terms and conditions of the BSD License

   which accompanies this distribution.  The full text of the license may be found at

@@ -85,13 +85,8 @@ MISC_SMBIOS_TABLE_FUNCTION(MiscSystemSlotDesignation)
   //  

   // Now we have got the full smbios record, call smbios protocol to add this record.

   //

-  SmbiosHandle = 0;

-  Status = Smbios-> Add(

-                      Smbios, 

-                      NULL,

-                      &SmbiosHandle, 

-                      (EFI_SMBIOS_TABLE_HEADER *) SmbiosRecord

-                      );

+  Status = AddSmbiosRecord (Smbios, &SmbiosHandle, (EFI_SMBIOS_TABLE_HEADER *) SmbiosRecord);

+

   FreePool(SmbiosRecord);

   return Status;

 }