IntelFrameworkModulePkg: LegacyBootMaintUiLib: remove set but unused variables

[mirror_edk2.git] / IntelFrameworkModulePkg / Library / LegacyBootMaintUiLib / LegacyBootMaintUi.c

diff --git a/IntelFrameworkModulePkg/Library/LegacyBootMaintUiLib/LegacyBootMaintUi.c b/IntelFrameworkModulePkg/Library/LegacyBootMaintUiLib/LegacyBootMaintUi.c
index 4ebba337deddc537351d8d25e61d6bf4508e0e40..3a3eeed3e8c6c3643098bf44dd54c01c4a10134c 100644 (file)
--- a/IntelFrameworkModulePkg/Library/LegacyBootMaintUiLib/LegacyBootMaintUi.c
+++ b/IntelFrameworkModulePkg/Library/LegacyBootMaintUiLib/LegacyBootMaintUi.c
@@ -1,7 +1,7 @@
 /** @file\r
   Legacy Boot Maintainence UI implementation.\r
 \r
-Copyright (c) 2004 - 2015, Intel Corporation. All rights reserved.<BR>\r
+Copyright (c) 2004 - 2016, Intel Corporation. All rights reserved.<BR>\r
 This program and the accompanying materials\r
 are licensed and made available under the terms and conditions of the BSD License\r
 which accompanies this distribution.  The full text of the license may be found at\r
@@ -109,8 +109,11 @@ HII_VENDOR_DEVICE_PATH  mLegacyBootOptionHiiVendorDevicePath = {
   @param EnBootOptionCount  Count of the enabled Boot Option Numbers\r
   @param DisBootOption      Callee allocated buffer containing the disabled Boot Option Numbers\r
   @param DisBootOptionCount Count of the disabled Boot Option Numbers\r
+\r
+  @return EFI_SUCCESS       The function completed successfully.\r
+  @retval other             Contain some error, details see  the status return by gRT->SetVariable.\r
 **/\r
-VOID\r
+EFI_STATUS\r
 OrderLegacyBootOption4SameType (\r
   UINT16                   *DevOrder,\r
   UINTN                    DevOrderCount,\r
@@ -143,6 +146,7 @@ OrderLegacyBootOption4SameType (
   *DisBootOptionCount = 0;\r
   *EnBootOptionCount  = 0;\r
   Index               = 0;\r
+  Status              = EFI_SUCCESS;\r
 \r
   ASSERT (BbsIndexArray != NULL);\r
   ASSERT (DeviceTypeArray != NULL);\r
@@ -207,11 +211,12 @@ OrderLegacyBootOption4SameType (
                   BootOrderSize,\r
                   BootOrder\r
                   );\r
-  ASSERT_EFI_ERROR (Status);\r
 \r
   FreePool (NewBootOption);\r
   FreePool (DeviceTypeArray);\r
   FreePool (BbsIndexArray);\r
+\r
+  return Status;\r
 }\r
 \r
 /**\r
@@ -222,8 +227,9 @@ OrderLegacyBootOption4SameType (
   @param NVMapData   The data for egacy BBS boot.\r
 \r
   @return EFI_SUCCESS           The function completed successfully.\r
-  @retval EFI_NOT_FOUND         If L"LegacyDevOrder" and gEfiLegacyDevOrderVariableGuid EFI Variable can be found.\r
+  @retval EFI_NOT_FOUND         If L"LegacyDevOrder" and gEfiLegacyDevOrderVariableGuid EFI Variable can not be found.\r
   @retval EFI_OUT_OF_RESOURCES  Fail to allocate memory resource\r
+  @retval other                 Contain some error, details see  the status return by gRT->SetVariable.\r
 **/\r
 EFI_STATUS\r
 UpdateBBSOption (\r
@@ -260,6 +266,8 @@ UpdateBBSOption (
   DisMap              = NULL;\r
   NewOrder            = NULL;\r
   CurrentType         = 0;\r
+  EnBootOption        = NULL;\r
+  DisBootOption       = NULL;\r
 \r
   \r
   DisMap  = mLegacyBootOptionPrivate->MaintainMapData->DisableMap;\r
@@ -381,7 +389,7 @@ UpdateBBSOption (
     //\r
     ASSERT (OptionMenu->MenuNumber == DevOrder->Length / sizeof (UINT16) - 1);\r
 \r
-    OrderLegacyBootOption4SameType (\r
+    Status = OrderLegacyBootOption4SameType (\r
       DevOrder->Data,\r
       DevOrder->Length / sizeof (UINT16) - 1,\r
       &EnBootOption,\r
@@ -389,6 +397,9 @@ UpdateBBSOption (
       &DisBootOption,\r
       &DisBootOptionCount\r
       );\r
+     if (EFI_ERROR(Status)) {\r
+       goto Fail;\r
+     }\r
 \r
     //\r
     // 2. Deactivate the DisBootOption and activate the EnBootOption\r
@@ -446,6 +457,15 @@ UpdateBBSOption (
                   OriginalPtr\r
                   );\r
 \r
+Fail:\r
+  if (EnBootOption != NULL) {\r
+    FreePool (EnBootOption);\r
+  }\r
+\r
+  if (DisBootOption != NULL) {\r
+    FreePool (DisBootOption);\r
+  }\r
+\r
   FreePool (OriginalPtr);\r
   return Status;\r
 }\r
@@ -700,7 +720,6 @@ UpdateLegacyDeviceOrderPage (
   LEGACY_MENU_ENTRY           *NewMenuEntry;\r
   EFI_STRING_ID               StrRef;\r
   EFI_STRING_ID               StrRefHelp;\r
-  BBS_TYPE                    BbsType;\r
   UINT16                      *Default;\r
   UINT16                      Index;\r
   UINT16                      Key;\r
@@ -718,7 +737,6 @@ UpdateLegacyDeviceOrderPage (
   TypeStr     = NULL;\r
   TypeStrHelp = NULL;\r
   Default     = NULL;\r
-  BbsType     = BBS_FLOPPY;\r
 \r
   RefreshLegacyUpdateData();\r
 \r
@@ -732,7 +750,6 @@ UpdateLegacyDeviceOrderPage (
     TypeStr     = STR_FLOPPY;\r
     TypeStrHelp = STR_FLOPPY_HELP;\r
     FormTitle   = STR_FLOPPY_TITLE;\r
-    BbsType     = BBS_FLOPPY;\r
     Default     = mLegacyBootOptionPrivate->MaintainMapData->CurrentNvData.LegacyFD;\r
     break;\r
 \r
@@ -742,7 +759,6 @@ UpdateLegacyDeviceOrderPage (
     TypeStr     = STR_HARDDISK;\r
     TypeStrHelp = STR_HARDDISK_HELP;\r
     FormTitle   = STR_HARDDISK_TITLE;\r
-    BbsType     = BBS_HARDDISK;\r
     Default     = mLegacyBootOptionPrivate->MaintainMapData->CurrentNvData.LegacyHD;\r
     break;\r
 \r
@@ -752,7 +768,6 @@ UpdateLegacyDeviceOrderPage (
     TypeStr     = STR_CDROM;\r
     TypeStrHelp = STR_CDROM_HELP;\r
     FormTitle   = STR_CDROM_TITLE;\r
-    BbsType     = BBS_CDROM;\r
     Default     = mLegacyBootOptionPrivate->MaintainMapData->CurrentNvData.LegacyCD;\r
     break;\r
 \r
@@ -762,7 +777,6 @@ UpdateLegacyDeviceOrderPage (
     TypeStr     = STR_NET;\r
     TypeStrHelp = STR_NET_HELP;\r
     FormTitle   = STR_NET_TITLE;\r
-    BbsType     = BBS_EMBED_NETWORK;\r
     Default     = mLegacyBootOptionPrivate->MaintainMapData->CurrentNvData.LegacyNET;\r
     break;\r
 \r
@@ -772,7 +786,6 @@ UpdateLegacyDeviceOrderPage (
     TypeStr     = STR_BEV;\r
     TypeStrHelp = STR_BEV_HELP;\r
     FormTitle   = STR_BEV_TITLE;\r
-    BbsType     = BBS_BEV_DEVICE;\r
     Default     = mLegacyBootOptionPrivate->MaintainMapData->CurrentNvData.LegacyBEV;\r
     break;\r
 \r
@@ -879,8 +892,6 @@ AdjustOptionValue (
   )\r
 {\r
   UINTN                       Number;\r
-  BBS_TYPE                    BbsType;\r
-  LEGACY_DEV_ORDER_ENTRY      *DevOrder;\r
   UINT16                      *Default;\r
   LEGACY_BOOT_NV_DATA         *CurrentNVMap;\r
   UINT16                      *CurrentVal;\r
@@ -895,12 +906,11 @@ AdjustOptionValue (
   UINTN                       Bit;\r
 \r
   Number = 0;\r
-  BbsType = BBS_UNKNOWN;\r
   CurrentVal = 0;\r
-  DevOrder = NULL;\r
   Default = NULL;\r
   NewValue = 0;\r
   NewValuePos = 0;\r
+  OldValue = 0;\r
 \r
   //\r
   // Update Select FD/HD/CD/NET/BEV Order Form\r
@@ -913,27 +923,22 @@ AdjustOptionValue (
 \r
   if (QuestionId >= LEGACY_FD_QUESTION_ID && QuestionId < LEGACY_FD_QUESTION_ID + MAX_MENU_NUMBER) {\r
     Number      = (UINT16) LegacyFDMenu.MenuNumber;\r
-    BbsType     = BBS_FLOPPY;\r
     CurrentVal  = CurrentNVMap->LegacyFD;\r
     Default     = mLegacyBootOptionPrivate->MaintainMapData->LastTimeNvData.LegacyFD;\r
   } else if (QuestionId >= LEGACY_HD_QUESTION_ID && QuestionId < LEGACY_HD_QUESTION_ID + MAX_MENU_NUMBER) {\r
     Number      = (UINT16) LegacyHDMenu.MenuNumber;\r
-    BbsType     = BBS_HARDDISK;\r
     CurrentVal  = CurrentNVMap->LegacyHD;\r
     Default     = mLegacyBootOptionPrivate->MaintainMapData->LastTimeNvData.LegacyHD;\r
   } else if (QuestionId >= LEGACY_CD_QUESTION_ID && QuestionId < LEGACY_CD_QUESTION_ID + MAX_MENU_NUMBER) {\r
     Number      = (UINT16) LegacyCDMenu.MenuNumber;\r
-    BbsType     = BBS_CDROM;\r
     CurrentVal  = CurrentNVMap->LegacyCD;\r
     Default     = mLegacyBootOptionPrivate->MaintainMapData->LastTimeNvData.LegacyCD;\r
   } else if (QuestionId >= LEGACY_NET_QUESTION_ID && QuestionId < LEGACY_NET_QUESTION_ID + MAX_MENU_NUMBER) {\r
     Number      = (UINT16) LegacyNETMenu.MenuNumber;\r
-    BbsType     = BBS_EMBED_NETWORK;\r
     CurrentVal  = CurrentNVMap->LegacyNET;\r
     Default     = mLegacyBootOptionPrivate->MaintainMapData->LastTimeNvData.LegacyNET;\r
   } else if (QuestionId >= LEGACY_BEV_QUESTION_ID && QuestionId < LEGACY_BEV_QUESTION_ID + MAX_MENU_NUMBER) {\r
     Number      = (UINT16) LegacyBEVMenu.MenuNumber;\r
-    BbsType     = BBS_BEV_DEVICE;\r
     CurrentVal  = CurrentNVMap->LegacyBEV;\r
     Default     = mLegacyBootOptionPrivate->MaintainMapData->LastTimeNvData.LegacyBEV;\r
   }\r