]> git.proxmox.com Git - mirror_edk2.git/blobdiff - ShellPkg/Library/UefiShellInstall1CommandsLib/Bcfg.c
Fix GCC build failure.
[mirror_edk2.git] / ShellPkg / Library / UefiShellInstall1CommandsLib / Bcfg.c
index 17c099d533defa707341d626be0d5ef5c51abce7..543ae376e9c8a2c6372107292f951d6a1c419015 100644 (file)
@@ -1,7 +1,7 @@
 /** @file\r
-  Main file for bcfg shell install1 function.\r
+  Main file for bcfg shell Install1 function.\r
 \r
-  Copyright (c) 2010, Intel Corporation. All rights reserved.<BR>\r
+  Copyright (c) 2010 - 2013, 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
 #include <Library/DevicePathLib.h>\r
 \r
 typedef enum {\r
-  BCFG_TARGET_BOOT_ORDER    = 0,\r
-  BCFG_TARGET_DRIVER_ORDER  = 1,\r
-  BCFG_TARGET_MAX           = 2\r
+  BcfgTargetBootOrder    = 0,\r
+  BcfgTargetDriverOrder  = 1,\r
+  BcfgTargetMax          = 2\r
 } BCFG_OPERATION_TARGET;\r
 \r
 typedef enum {\r
-  BCFG_TYPE_DUMP       = 0,\r
-  BCFG_TYPE_ADD        = 1,\r
-  BCFG_TYPE_ADDP       = 2,\r
-  BCFG_TYPE_ADDH       = 3,\r
-  BCFG_TYPE_RM         = 4,\r
-  BCFG_TYPE_MV         = 5,\r
-  BCFG_TYPE_OPT        = 6,\r
-  BCFG_TYPE_MAX        = 7\r
+  BcfgTypeDump       = 0,\r
+  BcfgTypeAdd        = 1,\r
+  BcfgTypeAddp       = 2,\r
+  BcfgTypeAddh       = 3,\r
+  BcfgTypeRm         = 4,\r
+  BcfgTypeMv         = 5,\r
+  BcfgTypeOpt        = 6,\r
+  BcfgTypeMax        = 7\r
 } BCFG_OPERATION_TYPE;\r
 \r
 typedef struct {\r
@@ -48,42 +48,237 @@ typedef struct {
 } BGFG_OPERATION;\r
 \r
 /**\r
-  Function to update the optional data associated with an option.\r
+  Get the actual number of entries in EFI_KEY_OPTION.Keys, from 0-3.\r
 \r
-  @param[in] Target     The type being modified.  BOOT or DRIVER\r
-  @param[in] OptData    The pointer to the data to modify with.\r
+  @param   KeyOption  Pointer to the EFI_KEY_OPTION structure. \r
 \r
-  @retval SHELL_SUCCESS The optional data was updated sucessfully.\r
+  @return  Actual number of entries in EFI_KEY_OPTION.Keys.\r
 **/\r
-SHELL_STATUS\r
+#define KEY_OPTION_INPUT_KEY_COUNT(KeyOption) \\r
+  (((KeyOption)->KeyData & EFI_KEY_OPTION_INPUT_KEY_COUNT_MASK) >> LowBitSet32 (EFI_KEY_OPTION_INPUT_KEY_COUNT_MASK))\r
+\r
+/**\r
+  Update the optional data for a boot or driver option.\r
+\r
+  If optional data exists it will be changed.\r
+\r
+  @param[in]      Index     The boot or driver option index update.\r
+  @param[in]      DataSize  The size in bytes of Data.\r
+  @param[in]      Data      The buffer for the optioanl data.\r
+  @param[in]      Target    The target of the operation.\r
+\r
+  @retval EFI_SUCCESS       The data was sucessfully updated.\r
+  @retval other             A error occured.\r
+**/\r
+EFI_STATUS\r
 EFIAPI\r
-BcfgAddOptInstall(\r
-  IN CONST BCFG_OPERATION_TARGET  Target,\r
-  IN CONST CHAR16 *OptData\r
+UpdateOptionalData(\r
+  UINT16                          Index, \r
+  UINTN                           DataSize, \r
+  UINT8                           *Data,\r
+  IN CONST BCFG_OPERATION_TARGET  Target\r
   )\r
 {\r
-  ShellPrintEx(-1, -1, L"use of -opt is not currently supported.");\r
-  return (SHELL_UNSUPPORTED);\r
+  EFI_STATUS  Status;\r
+  CHAR16      VariableName[12];\r
+  UINTN       OriginalSize;\r
+  UINT8       *OriginalData;\r
+  UINTN       NewSize;\r
+  UINT8       *NewData;\r
+  UINTN       OriginalOptionDataSize;\r
+\r
+  UnicodeSPrint(VariableName, sizeof(VariableName), L"%s%04x", Target == BcfgTargetBootOrder?L"Boot":L"Driver", Index);\r
+  \r
+  OriginalSize = 0;\r
+  OriginalData = NULL;\r
+  NewData      = NULL;\r
+  NewSize      = 0;\r
+\r
+  Status = gRT->GetVariable(\r
+      VariableName,\r
+      (EFI_GUID*)&gEfiGlobalVariableGuid,\r
+      NULL,\r
+      &OriginalSize,\r
+      OriginalData);\r
+  if (Status == EFI_BUFFER_TOO_SMALL) {\r
+    OriginalData = AllocateZeroPool(OriginalSize);\r
+    if (OriginalData == NULL) {\r
+      return (EFI_OUT_OF_RESOURCES);\r
+    }\r
+    Status = gRT->GetVariable(\r
+        VariableName,\r
+        (EFI_GUID*)&gEfiGlobalVariableGuid,\r
+        NULL,\r
+        &OriginalSize,\r
+        OriginalData);\r
+  }\r
+\r
+  if (!EFI_ERROR(Status)) {\r
+    //\r
+    // Allocate new struct and discard old optional data.\r
+    //\r
+    ASSERT (OriginalData != NULL);\r
+    OriginalOptionDataSize  = sizeof(UINT32) + sizeof(UINT16) + StrSize(((CHAR16*)(OriginalData + sizeof(UINT32) + sizeof(UINT16))));\r
+    OriginalOptionDataSize += (*(UINT16*)(OriginalData + sizeof(UINT32)));\r
+    OriginalOptionDataSize -= OriginalSize;\r
+    NewSize = OriginalSize - OriginalOptionDataSize + DataSize;\r
+    NewData = AllocateCopyPool(NewSize, OriginalData);\r
+    if (NewData == NULL) {\r
+      Status = EFI_OUT_OF_RESOURCES;\r
+    } else {\r
+      CopyMem(NewData + OriginalSize - OriginalOptionDataSize, Data, DataSize);\r
+    }\r
+  }\r
+\r
+  if (!EFI_ERROR(Status)) {\r
+    //\r
+    // put the data back under the variable\r
+    //\r
+    Status = gRT->SetVariable(\r
+      VariableName, \r
+      (EFI_GUID*)&gEfiGlobalVariableGuid,\r
+      EFI_VARIABLE_NON_VOLATILE|EFI_VARIABLE_BOOTSERVICE_ACCESS|EFI_VARIABLE_RUNTIME_ACCESS,\r
+      NewSize,\r
+      NewData);\r
+  }\r
+\r
+  SHELL_FREE_NON_NULL(OriginalData);\r
+  SHELL_FREE_NON_NULL(NewData);\r
+  return (Status);\r
 }\r
 \r
 /**\r
-  Function to add an option.\r
-\r
-  @param[in] Position     The position to add this in at.\r
-  @param[in] File         The file to add as the option.\r
-  @param[in] Desc         The description.\r
-  @param[in] CurrentOrder The current order of that type.\r
-  @param[in] OrderCount   The number of items in the current order.\r
-  @param[in] Target       The type being modified.  BOOT or DRIVER\r
-  @param[in] UseHandle    Add something by a handle.  Uses HandleNumber if TRUE and File if FALSE.\r
-  @param[in] UsePath      Add something by local path.  Only used of UseHandle is FALSE.\r
-  @param[in] HandleNumber The HandleIndex to use.\r
-\r
-  @retval SHELL_SUCCESS The option was added sucessfully.\r
+  This function will get a CRC for a boot option.\r
+\r
+  @param[in, out] Crc         The CRC value to return.\r
+  @param[in]      BootIndex   The boot option index to CRC.\r
+\r
+  @retval EFI_SUCCESS           The CRC was sucessfully returned.\r
+  @retval other                 A error occured.\r
+**/\r
+EFI_STATUS\r
+EFIAPI\r
+GetBootOptionCrc(\r
+  UINT32      *Crc, \r
+  UINT16      BootIndex\r
+  )\r
+{\r
+  CHAR16      VariableName[12];\r
+  EFI_STATUS  Status;\r
+  UINT8       *Buffer;\r
+  UINTN       BufferSize;\r
+\r
+  Buffer      = NULL;\r
+  BufferSize  = 0;\r
+\r
+  //\r
+  // Get the data Buffer\r
+  //\r
+  UnicodeSPrint(VariableName, sizeof(VariableName), L"%Boot%04x", BootIndex);\r
+  Status = gRT->GetVariable(\r
+      VariableName,\r
+      (EFI_GUID*)&gEfiGlobalVariableGuid,\r
+      NULL,\r
+      &BufferSize,\r
+      NULL);\r
+  if (Status == EFI_BUFFER_TOO_SMALL) {\r
+    Buffer = AllocateZeroPool(BufferSize);\r
+    Status = gRT->GetVariable(\r
+        VariableName,\r
+        (EFI_GUID*)&gEfiGlobalVariableGuid,\r
+        NULL,\r
+        &BufferSize,\r
+        Buffer);\r
+  }\r
+\r
+  //\r
+  // Get the CRC computed\r
+  //\r
+  if (!EFI_ERROR(Status)) {\r
+    Status = gBS->CalculateCrc32 (Buffer, BufferSize, Crc);\r
+  }\r
+\r
+  SHELL_FREE_NON_NULL(Buffer);\r
+  return EFI_SUCCESS;\r
+}\r
+\r
+/**\r
+  This function will populate the device path protocol parameter based on TheHandle.\r
+\r
+  @param[in]      TheHandle     Driver handle.\r
+  @param[in, out] FilePath      On a sucessful return the device path to the handle.\r
+\r
+  @retval EFI_SUCCESS           The device path was sucessfully returned.\r
+  @retval other                 A error from gBS->HandleProtocol.\r
+\r
+  @sa HandleProtocol\r
+**/\r
+EFI_STATUS\r
+EFIAPI\r
+GetDevicePathForDriverHandleInstall1 (\r
+  IN EFI_HANDLE                   TheHandle,\r
+  IN OUT EFI_DEVICE_PATH_PROTOCOL **FilePath\r
+  )\r
+{\r
+  EFI_STATUS                Status;\r
+  EFI_LOADED_IMAGE_PROTOCOL *LoadedImage;\r
+  EFI_DEVICE_PATH_PROTOCOL  *ImageDevicePath;\r
+\r
+  Status = gBS->OpenProtocol (\r
+                TheHandle,\r
+                &gEfiLoadedImageProtocolGuid,\r
+                (VOID**)&LoadedImage,\r
+                gImageHandle,\r
+                NULL,\r
+                EFI_OPEN_PROTOCOL_GET_PROTOCOL\r
+               );\r
+  if (!EFI_ERROR (Status)) {\r
+    Status = gBS->OpenProtocol (\r
+                  LoadedImage->DeviceHandle,\r
+                  &gEfiDevicePathProtocolGuid,\r
+                  (VOID**)&ImageDevicePath,\r
+                  gImageHandle,\r
+                  NULL,\r
+                  EFI_OPEN_PROTOCOL_GET_PROTOCOL\r
+                 );\r
+    if (!EFI_ERROR (Status)) {\r
+//      *DevPath  = DuplicateDevicePath (ImageDevicePath);\r
+//      *FilePath = DuplicateDevicePath (LoadedImage->FilePath);\r
+        *FilePath = AppendDevicePath(ImageDevicePath,LoadedImage->FilePath);\r
+      gBS->CloseProtocol(\r
+                  LoadedImage->DeviceHandle,\r
+                  &gEfiDevicePathProtocolGuid,\r
+                  gImageHandle,\r
+                  NULL);\r
+    }\r
+    gBS->CloseProtocol(\r
+                TheHandle,\r
+                &gEfiLoadedImageProtocolGuid,\r
+                gImageHandle,\r
+                NULL);\r
+  }\r
+  return (Status);\r
+}\r
+\r
+/**\r
+  Function to add a option.\r
+\r
+  @param[in] Position       The position to add Target at.\r
+  @param[in] File           The file to make the target.\r
+  @param[in] Desc           The description text.\r
+  @param[in] CurrentOrder   The pointer to the current order of items.\r
+  @param[in] OrderCount     The number if items in CurrentOrder.\r
+  @param[in] Target         The info on the option to add.\r
+  @param[in] UseHandle      TRUE to use HandleNumber, FALSE to use File and Desc.\r
+  @param[in] UsePath        TRUE to convert to devicepath.\r
+  @param[in] HandleNumber   The handle number to add.\r
+\r
+  @retval SHELL_SUCCESS             The operation was successful.\r
+  @retval SHELL_INVALID_PARAMETER   A parameter was invalid.\r
 **/\r
 SHELL_STATUS\r
 EFIAPI\r
-BcfgAddInstall (\r
+BcfgAddInstall1(\r
   IN       UINTN                  Position,\r
   IN CONST CHAR16                 *File,\r
   IN CONST CHAR16                 *Desc,\r
@@ -91,16 +286,16 @@ BcfgAddInstall (
   IN CONST UINTN                  OrderCount,\r
   IN CONST BCFG_OPERATION_TARGET  Target,\r
   IN CONST BOOLEAN                UseHandle,\r
-  IN CONST BOOLEAN                UseFullPath,\r
+  IN CONST BOOLEAN                UsePath,\r
   IN CONST UINTN                  HandleNumber\r
   )\r
 {\r
   EFI_STATUS                Status;\r
-  EFI_DEVICE_PATH_PROTOCOL  *DevicePath, *FilePath, *FileNode;\r
+  EFI_DEVICE_PATH_PROTOCOL  *DevicePath;\r
+  EFI_DEVICE_PATH_PROTOCOL  *FilePath;\r
   CHAR16                    *Str;\r
-  CONST CHAR16              *p;\r
-  UINT8                     *p8;\r
-  UINT8                     *p8Copy;\r
+  UINT8                     *TempByteBuffer;\r
+  UINT8                     *TempByteStart;\r
   EFI_SHELL_FILE_INFO       *Arg;\r
   EFI_SHELL_FILE_INFO       *FileList;\r
   CHAR16                    OptionStr[40];\r
@@ -110,50 +305,106 @@ BcfgAddInstall (
   UINTN                     Index;\r
   EFI_HANDLE                *Handles;\r
   EFI_HANDLE                CurHandle;\r
+  UINTN                     DriverBindingHandleCount;\r
+  UINTN                     ParentControllerHandleCount;\r
+  UINTN                     ChildControllerHandleCount;\r
   SHELL_STATUS              ShellStatus;\r
   UINT16                    *NewOrder;\r
-  EFI_LOADED_IMAGE_PROTOCOL *Image;\r
 \r
   if (!UseHandle) {\r
-    ASSERT(File != NULL);\r
-    ASSERT(Desc != NULL);\r
+    if (File == NULL || Desc == NULL) {\r
+      return (SHELL_INVALID_PARAMETER);\r
+    }\r
   } else {\r
-    ASSERT(HandleNumber != 0);\r
+    if (HandleNumber == 0) {\r
+      return (SHELL_INVALID_PARAMETER);\r
+    }\r
   }\r
 \r
-  Str                         = NULL;\r
-  FilePath                    = NULL;\r
-  FileNode                    = NULL;\r
-  FileList                    = NULL;\r
-  Handles                     = NULL;\r
-  ShellStatus                 = SHELL_SUCCESS;\r
-  TargetLocation              = 0xFFFF;\r
+  if (Position > OrderCount) {\r
+    Position =  OrderCount;\r
+  }\r
+\r
+  Str             = NULL;\r
+  FilePath        = NULL;\r
+  FileList        = NULL;\r
+  Handles         = NULL;\r
+  ShellStatus     = SHELL_SUCCESS;\r
+  TargetLocation  = 0xFFFF;\r
 \r
   if (UseHandle) {\r
     CurHandle = ConvertHandleIndexToHandle(HandleNumber);\r
     if (CurHandle == NULL) {\r
-      ShellPrintHiiEx(-1, -1, NULL, STRING_TOKEN (STR_GEN_PROBLEM), gShellInstall1HiiHandle, L"<HandleNumber>");\r
+      ShellPrintHiiEx(-1, -1, NULL, STRING_TOKEN (STR_GEN_PROBLEM), gShellInstall1HiiHandle, L"Handle Number");\r
       ShellStatus = SHELL_INVALID_PARAMETER;\r
     } else {\r
-      FilePath = NULL;\r
-      Status = gBS->HandleProtocol (CurHandle, &gEfiLoadedImageDevicePathProtocolGuid, (VOID**)&FilePath);\r
-      if (EFI_ERROR (Status)) {\r
-        Status = EFI_SUCCESS;\r
+      if (Target == BcfgTargetBootOrder) {\r
         //\r
-        // Try to construct the DevicePath\r
+        //Make sure that the handle should point to a real controller\r
         //\r
-        Status = gBS->OpenProtocol(CurHandle, &gEfiLoadedImageProtocolGuid, (VOID**)&Image, gImageHandle, NULL, EFI_OPEN_PROTOCOL_GET_PROTOCOL);\r
-        if (EFI_ERROR(Status)) {\r
+        Status = PARSE_HANDLE_DATABASE_UEFI_DRIVERS (\r
+                   CurHandle,\r
+                   &DriverBindingHandleCount,\r
+                   NULL);\r
+\r
+        Status = PARSE_HANDLE_DATABASE_PARENTS (\r
+                   CurHandle,\r
+                   &ParentControllerHandleCount,\r
+                   NULL);\r
+\r
+        Status = ParseHandleDatabaseForChildControllers (\r
+                   CurHandle,\r
+                   &ChildControllerHandleCount,\r
+                   NULL);\r
+\r
+        if (DriverBindingHandleCount > 0\r
+              || ParentControllerHandleCount > 0\r
+              || ChildControllerHandleCount > 0) {\r
+          FilePath = NULL;\r
+          Status = gBS->HandleProtocol (\r
+                     CurHandle,\r
+                     &gEfiDevicePathProtocolGuid,\r
+                     (VOID**)&FilePath);\r
+        }\r
+        if (EFI_ERROR (Status)) {\r
           ShellPrintHiiEx(-1, -1, NULL, STRING_TOKEN (STR_BCFG_HANDLE), gShellInstall1HiiHandle, HandleNumber);\r
           ShellStatus = SHELL_INVALID_PARAMETER;\r
+        }\r
+      } else {\r
+        //\r
+        //Make sure that the handle should point to driver, not a controller.\r
+        //\r
+        Status = PARSE_HANDLE_DATABASE_UEFI_DRIVERS (\r
+                   CurHandle,\r
+                   &DriverBindingHandleCount,\r
+                   NULL);\r
+\r
+        Status = PARSE_HANDLE_DATABASE_PARENTS (\r
+                   CurHandle,\r
+                   &ParentControllerHandleCount,\r
+                   NULL);\r
+\r
+        Status = ParseHandleDatabaseForChildControllers (\r
+                   CurHandle,\r
+                   &ChildControllerHandleCount,\r
+                   NULL);\r
+\r
+        Status = gBS->HandleProtocol (\r
+                   CurHandle,\r
+                   &gEfiDevicePathProtocolGuid,\r
+                   (VOID**)&FilePath);\r
+\r
+        if (DriverBindingHandleCount > 0\r
+              || ParentControllerHandleCount > 0\r
+              || ChildControllerHandleCount > 0\r
+              || !EFI_ERROR(Status) ) {\r
+          ShellPrintHiiEx(-1, -1, NULL, STRING_TOKEN (STR_GEN_PROBLEM), gShellInstall1HiiHandle, L"Handle Number");\r
+          ShellStatus = SHELL_INVALID_PARAMETER;\r
         } else {\r
-          Status = gBS->HandleProtocol (Image->DeviceHandle, &gEfiDevicePathProtocolGuid, (VOID**)&DevicePath);\r
-          if (EFI_ERROR(Status)) {\r
-            ShellPrintHiiEx(-1, -1, NULL, STRING_TOKEN (STR_BCFG_HANDLE), gShellInstall1HiiHandle, HandleNumber);\r
-            ShellStatus = SHELL_INVALID_PARAMETER;\r
-          } else {\r
-            FilePath = AppendDevicePath(DevicePath, Image->FilePath);\r
-          }\r
+          //\r
+          // Get the DevicePath from the loaded image information.\r
+          //\r
+          Status = GetDevicePathForDriverHandleInstall1(CurHandle, &FilePath);\r
         }\r
       }\r
     }\r
@@ -161,11 +412,15 @@ BcfgAddInstall (
     //\r
     // Get file info\r
     //\r
-    Status = ShellOpenFileMetaArg ((CHAR16*)File, EFI_FILE_MODE_READ, &FileList);\r
-    if (EFI_ERROR(Status)) {\r
+    ShellOpenFileMetaArg ((CHAR16*)File, EFI_FILE_MODE_READ, &FileList);\r
+\r
+    if (FileList == NULL) {\r
+      //\r
+      // If filename matched nothing fail\r
+      //\r
       ShellPrintHiiEx(-1, -1, NULL, STRING_TOKEN (STR_GEN_FILE_OPEN_FAIL), gShellInstall1HiiHandle, File);\r
-      ShellStatus = SHELL_NOT_FOUND;\r
-    } else if (FileList == NULL || FileList->Link.ForwardLink != FileList->Link.BackLink) {\r
+      ShellStatus = SHELL_INVALID_PARAMETER;\r
+    } else if (FileList->Link.ForwardLink != FileList->Link.BackLink) {\r
       //\r
       // If filename expanded to multiple names, fail\r
       //\r
@@ -184,18 +439,39 @@ BcfgAddInstall (
         //\r
         // get the device path\r
         //\r
-        DevicePath = mEfiShellProtocol->GetDevicePathFromFilePath(Arg->FullName);\r
+        DevicePath = gEfiShellProtocol->GetDevicePathFromFilePath(Arg->FullName);\r
         if (DevicePath == NULL) {\r
           ShellPrintHiiEx(-1, -1, NULL, STRING_TOKEN (STR_BCFG_FILE_DP), gShellInstall1HiiHandle, Arg->FullName);\r
           ShellStatus = SHELL_UNSUPPORTED;\r
         } else {\r
-          if (UseFullPath) {\r
-            FilePath = DuplicateDevicePath(DevicePath);\r
+/*\r
+          if (UsePath) {\r
+            DevPath = DevicePath;\r
+            while (!IsDevicePathEnd(DevPath)) {\r
+              if ((DevicePathType(DevPath) == MEDIA_DEVICE_PATH) &&\r
+                (DevicePathSubType(DevPath) == MEDIA_HARDDRIVE_DP)) {\r
+\r
+                //\r
+                // If we find it use it instead\r
+                //\r
+                DevicePath = DevPath;\r
+                break;\r
+              }\r
+              DevPath = NextDevicePathNode(DevPath);\r
+            }\r
+            //\r
+            // append the file\r
+            //\r
+            for(StringWalker=Arg->FullName; *StringWalker != CHAR_NULL && *StringWalker != ':'; StringWalker++);\r
+            FileNode = FileDevicePath(NULL, StringWalker+1);\r
+            FilePath = AppendDevicePath(DevicePath, FileNode);\r
+            FreePool(FileNode);\r
           } else {\r
-            for(p=Arg->FullName; *p != CHAR_NULL && *p != ':'; p++);\r
-            FilePath = FileDevicePath(NULL, p+1);\r
+*/\r
+            FilePath = DuplicateDevicePath(DevicePath);\r
+/*\r
           }\r
-\r
+*/\r
           FreePool(DevicePath);\r
         }\r
       }\r
@@ -236,72 +512,71 @@ BcfgAddInstall (
     DescSize = StrSize(Desc);\r
     FilePathSize = GetDevicePathSize (FilePath);\r
 \r
-    p8 = AllocateZeroPool(sizeof(UINT32) + sizeof(UINT16) + DescSize + FilePathSize);\r
-    if (p8 != NULL) {\r
-      p8Copy = p8;\r
-      *((UINT32 *) p8) = LOAD_OPTION_ACTIVE;      // Attributes\r
-      p8 += sizeof (UINT32);\r
+    TempByteBuffer = AllocateZeroPool(sizeof(UINT32) + sizeof(UINT16) + DescSize + FilePathSize);\r
+    if (TempByteBuffer != NULL) {\r
+      TempByteStart  = TempByteBuffer;\r
+      *((UINT32 *) TempByteBuffer) = LOAD_OPTION_ACTIVE;      // Attributes\r
+      TempByteBuffer += sizeof (UINT32);\r
 \r
-      *((UINT16 *) p8) = (UINT16)FilePathSize;    // FilePathListLength\r
-      p8 += sizeof (UINT16);\r
+      *((UINT16 *) TempByteBuffer) = (UINT16)FilePathSize;    // FilePathListLength\r
+      TempByteBuffer += sizeof (UINT16);\r
 \r
-      CopyMem (p8, Desc, DescSize);\r
-      p8 += DescSize;\r
-      CopyMem (p8, FilePath, FilePathSize);\r
+      CopyMem (TempByteBuffer, Desc, DescSize);\r
+      TempByteBuffer += DescSize;\r
+      ASSERT (FilePath != NULL);\r
+      CopyMem (TempByteBuffer, FilePath, FilePathSize);\r
 \r
-      UnicodeSPrint (OptionStr, sizeof(OptionStr), L"%s%04x", Target == BCFG_TARGET_BOOT_ORDER?L"Boot":L"Driver", TargetLocation);\r
+      UnicodeSPrint (OptionStr, sizeof(OptionStr), L"%s%04x", Target == BcfgTargetBootOrder?L"Boot":L"Driver", TargetLocation);\r
       Status = gRT->SetVariable (\r
             OptionStr,\r
             &gEfiGlobalVariableGuid,\r
             EFI_VARIABLE_NON_VOLATILE|EFI_VARIABLE_BOOTSERVICE_ACCESS|EFI_VARIABLE_RUNTIME_ACCESS,\r
             sizeof(UINT32) + sizeof(UINT16) + DescSize + FilePathSize,\r
-            p8Copy\r
+            TempByteStart\r
            );\r
 \r
-      FreePool(p8Copy);\r
+      FreePool(TempByteStart);\r
+    } else {\r
+      Status = EFI_OUT_OF_RESOURCES;\r
+    }\r
 \r
-      if (EFI_ERROR(Status)) {\r
-        ShellPrintHiiEx(-1, -1, NULL, STRING_TOKEN (STR_BCFG_SET_VAR_FAIL), gShellInstall1HiiHandle, OptionStr, Status);\r
-      } else {\r
-        NewOrder = AllocateZeroPool((OrderCount+1)*sizeof(UINT16));\r
-        ASSERT(NewOrder != NULL);\r
-        CopyMem(NewOrder, CurrentOrder, (OrderCount)*sizeof(UINT16));\r
+    if (EFI_ERROR(Status)) {\r
+      ShellPrintHiiEx(-1, -1, NULL, STRING_TOKEN (STR_BCFG_SET_VAR_FAIL), gShellInstall1HiiHandle, OptionStr, Status);\r
+    } else {\r
+      NewOrder = AllocateZeroPool((OrderCount+1)*sizeof(NewOrder[0]));\r
+      ASSERT(NewOrder != NULL);\r
+      CopyMem(NewOrder, CurrentOrder, (OrderCount)*sizeof(NewOrder[0]));\r
 \r
-        //\r
-        // Insert target into order list\r
-        //\r
-        for (Index=OrderCount; Index > Position; Index--) {\r
-          NewOrder[Index] = NewOrder[Index-1];\r
-        }\r
+      //\r
+      // Insert target into order list\r
+      //\r
+      for (Index=OrderCount; Index > Position; Index--) {\r
+        NewOrder[Index] = NewOrder[Index-1];\r
+      }\r
 \r
-        NewOrder[Index] = (UINT16)TargetLocation;\r
-        Status = gRT->SetVariable (\r
-          Target == BCFG_TARGET_BOOT_ORDER?L"BootOrder":L"DriverOrder",\r
-          &gEfiGlobalVariableGuid,\r
-          EFI_VARIABLE_NON_VOLATILE|EFI_VARIABLE_BOOTSERVICE_ACCESS|EFI_VARIABLE_RUNTIME_ACCESS,\r
-          (OrderCount+1) * sizeof(UINT16),\r
-          NewOrder\r
-         );\r
+      NewOrder[Position] = (UINT16) TargetLocation;\r
+      Status = gRT->SetVariable (\r
+        Target == BcfgTargetBootOrder?L"BootOrder":L"DriverOrder",\r
+        &gEfiGlobalVariableGuid,\r
+        EFI_VARIABLE_NON_VOLATILE|EFI_VARIABLE_BOOTSERVICE_ACCESS|EFI_VARIABLE_RUNTIME_ACCESS,\r
+        (OrderCount+1) * sizeof(UINT16),\r
+        NewOrder\r
+       );\r
 \r
-        FreePool(NewOrder);\r
+      FreePool(NewOrder);\r
 \r
-        if (EFI_ERROR(Status)) {\r
-          ShellPrintHiiEx(-1, -1, NULL, STRING_TOKEN (STR_BCFG_WRITE_FAIL), gShellInstall1HiiHandle, Target == BCFG_TARGET_BOOT_ORDER?L"BootOrder":L"DriverOrder", Status);\r
-          ShellStatus = SHELL_INVALID_PARAMETER;\r
-        } else {\r
-          ShellPrintEx (-1, -1, L"bcfg: Add %s as %x\n", OptionStr, Index);\r
-        }\r
+      if (EFI_ERROR(Status)) {\r
+        ShellPrintHiiEx(-1, -1, NULL, STRING_TOKEN (STR_BCFG_WRITE_FAIL), gShellInstall1HiiHandle, Target == BcfgTargetBootOrder?L"BootOrder":L"DriverOrder", Status);\r
+        ShellStatus = SHELL_INVALID_PARAMETER;\r
+      } else {\r
+        Print (L"bcfg: Add %s as %x\n", OptionStr, Position);\r
       }\r
-    } else {\r
-      ShellPrintHiiEx(-1, -1, NULL, STRING_TOKEN (STR_GEN_NO_MEM), gShellInstall1HiiHandle);\r
-      ShellStatus = SHELL_OUT_OF_RESOURCES;\r
     }\r
   }\r
 \r
 //\r
 //If always Free FilePath, will free devicepath in system when use "addh"\r
 //\r
-\r
   if (FilePath!=NULL && !UseHandle) {\r
     FreePool (FilePath);\r
   }\r
@@ -321,42 +596,32 @@ BcfgAddInstall (
   return (ShellStatus);\r
 }\r
 \r
+/**\r
+  Funciton to remove an item.\r
+\r
+  @param[in] Target         The target item to move.\r
+  @param[in] CurrentOrder   The pointer to the current order of items.\r
+  @param[in] OrderCount     The number if items in CurrentOrder.\r
+  @param[in] Location       The current location of the Target.\r
+\r
+  @retval SHELL_SUCCESS             The operation was successful.\r
+  @retval SHELL_INVALID_PARAMETER   A parameter was invalid.\r
+**/\r
 SHELL_STATUS\r
 EFIAPI\r
-BcfgRemoveInstall(\r
+BcfgRemoveInstall1(\r
   IN CONST BCFG_OPERATION_TARGET  Target,\r
   IN CONST UINT16                 *CurrentOrder,\r
   IN CONST UINTN                  OrderCount,\r
-  IN CONST UINT16                  Location\r
+  IN CONST UINT16                 Location\r
   )\r
 {\r
   CHAR16      VariableName[12];\r
   UINT16      *NewOrder;\r
   EFI_STATUS  Status;\r
+  UINTN       NewCount;\r
 \r
-  NewOrder  = AllocatePool(OrderCount*sizeof(CurrentOrder[0]));\r
-  if (NewOrder == NULL) {\r
-    return (SHELL_OUT_OF_RESOURCES);\r
-  }\r
-\r
-  CopyMem(NewOrder, CurrentOrder, OrderCount*sizeof(CurrentOrder[0]));\r
-  CopyMem(NewOrder+Location, NewOrder+Location+1, (OrderCount - Location - 1)*sizeof(CurrentOrder[0]));\r
-\r
-  Status = gRT->SetVariable(\r
-    Target == BCFG_TARGET_BOOT_ORDER?(CHAR16*)L"BootOrder":(CHAR16*)L"DriverOrder",\r
-    (EFI_GUID*)&gEfiGlobalVariableGuid,\r
-    EFI_VARIABLE_NON_VOLATILE|EFI_VARIABLE_BOOTSERVICE_ACCESS|EFI_VARIABLE_RUNTIME_ACCESS,\r
-    (OrderCount-1)*sizeof(CurrentOrder[0]), // drop 1 off since the list is 1 shorter\r
-    NewOrder);\r
-\r
-  FreePool(NewOrder);\r
-\r
-  if (EFI_ERROR(Status)) {\r
-    ShellPrintHiiEx(-1, -1, NULL, STRING_TOKEN (STR_BCFG_WRITE_FAIL), gShellInstall1HiiHandle, Target == BCFG_TARGET_BOOT_ORDER?(CHAR16*)L"BootOrder":(CHAR16*)L"DriverOrder", Status);\r
-    return (SHELL_INVALID_PARAMETER);\r
-  }\r
-\r
-  UnicodeSPrint(VariableName, sizeof(VariableName), L"%s%04x", Target == BCFG_TARGET_BOOT_ORDER?L"Boot":L"Driver", CurrentOrder[Location]);\r
+  UnicodeSPrint(VariableName, sizeof(VariableName), L"%s%04x", Target == BcfgTargetBootOrder?L"Boot":L"Driver", CurrentOrder[Location]);\r
   Status = gRT->SetVariable(\r
     VariableName,\r
     (EFI_GUID*)&gEfiGlobalVariableGuid,\r
@@ -367,13 +632,45 @@ BcfgRemoveInstall(
     ShellPrintHiiEx(-1, -1, NULL, STRING_TOKEN (STR_BCFG_WRITE_FAIL), gShellInstall1HiiHandle, VariableName, Status);\r
     return (SHELL_INVALID_PARAMETER);\r
   }\r
-\r
+  NewOrder = AllocateZeroPool(OrderCount*sizeof(CurrentOrder[0]));\r
+  if (NewOrder != NULL) {\r
+    NewCount = OrderCount;\r
+    CopyMem(NewOrder, CurrentOrder, OrderCount*sizeof(CurrentOrder[0]));\r
+    CopyMem(NewOrder+Location, NewOrder+Location+1, (OrderCount - Location - 1)*sizeof(CurrentOrder[0]));\r
+    NewCount--;\r
+\r
+    Status = gRT->SetVariable(\r
+      Target == BcfgTargetBootOrder?(CHAR16*)L"BootOrder":(CHAR16*)L"DriverOrder",\r
+      (EFI_GUID*)&gEfiGlobalVariableGuid,\r
+      EFI_VARIABLE_NON_VOLATILE|EFI_VARIABLE_BOOTSERVICE_ACCESS|EFI_VARIABLE_RUNTIME_ACCESS,\r
+      NewCount*sizeof(NewOrder[0]),\r
+      NewOrder);\r
+    FreePool(NewOrder);\r
+  } else {\r
+    Status = EFI_OUT_OF_RESOURCES;\r
+  }\r
+  if (EFI_ERROR(Status)) {\r
+    ShellPrintHiiEx(-1, -1, NULL, STRING_TOKEN (STR_BCFG_WRITE_FAIL), gShellInstall1HiiHandle, Target == BcfgTargetBootOrder?(CHAR16*)L"BootOrder":(CHAR16*)L"DriverOrder", Status);\r
+    return (SHELL_INVALID_PARAMETER);\r
+  }\r
   return (SHELL_SUCCESS);\r
 }\r
 \r
+/**\r
+  Funciton to move a item to another location.\r
+\r
+  @param[in] Target         The target item to move.\r
+  @param[in] CurrentOrder   The pointer to the current order of items.\r
+  @param[in] OrderCount     The number if items in CurrentOrder.\r
+  @param[in] OldLocation    The current location of the Target.\r
+  @param[in] NewLocation    The desired location of the Target.\r
+\r
+  @retval SHELL_SUCCESS             The operation was successful.\r
+  @retval SHELL_INVALID_PARAMETER   A parameter was invalid.\r
+**/\r
 SHELL_STATUS\r
 EFIAPI\r
-BcfgMoveInstall(\r
+BcfgMoveInstall1(\r
   IN CONST BCFG_OPERATION_TARGET  Target,\r
   IN CONST UINT16                 *CurrentOrder,\r
   IN CONST UINTN                  OrderCount,\r
@@ -385,22 +682,18 @@ BcfgMoveInstall(
   EFI_STATUS        Status;\r
   UINT16            Temp;\r
 \r
-  NewOrder = AllocatePool(OrderCount*sizeof(CurrentOrder[0]));\r
+  NewOrder = AllocateZeroPool(OrderCount*sizeof(CurrentOrder[0]));\r
   ASSERT(NewOrder != NULL);\r
 \r
   Temp = CurrentOrder[OldLocation];\r
   CopyMem(NewOrder, CurrentOrder, OrderCount*sizeof(CurrentOrder[0]));\r
   CopyMem(NewOrder+OldLocation, NewOrder+OldLocation+1, (OrderCount - OldLocation - 1)*sizeof(CurrentOrder[0]));\r
-  if (NewLocation == OrderCount) {\r
-    NewOrder[OrderCount-1] = Temp;\r
-  } else {\r
-    CopyMem(NewOrder+NewLocation+1, NewOrder+NewLocation, (OrderCount - NewLocation - 1)*sizeof(CurrentOrder[0]));\r
-    NewOrder[NewLocation] = Temp;\r
-  }\r
+  CopyMem(NewOrder+NewLocation+1, NewOrder+NewLocation, (OrderCount - NewLocation - 1)*sizeof(CurrentOrder[0]));\r
+  NewOrder[NewLocation] = Temp;\r
 \r
 \r
   Status = gRT->SetVariable(\r
-    Target == BCFG_TARGET_BOOT_ORDER?(CHAR16*)L"BootOrder":(CHAR16*)L"DriverOrder",\r
+    Target == BcfgTargetBootOrder?(CHAR16*)L"BootOrder":(CHAR16*)L"DriverOrder",\r
     (EFI_GUID*)&gEfiGlobalVariableGuid,\r
     EFI_VARIABLE_NON_VOLATILE|EFI_VARIABLE_BOOTSERVICE_ACCESS|EFI_VARIABLE_RUNTIME_ACCESS,\r
     OrderCount*sizeof(CurrentOrder[0]),\r
@@ -409,18 +702,309 @@ BcfgMoveInstall(
   FreePool(NewOrder);\r
 \r
   if (EFI_ERROR(Status)) {\r
-    ShellPrintHiiEx(-1, -1, NULL, STRING_TOKEN (STR_BCFG_WRITE_FAIL), gShellInstall1HiiHandle, Target == BCFG_TARGET_BOOT_ORDER?(CHAR16*)L"BootOrder":(CHAR16*)L"DriverOrder", Status);\r
+    ShellPrintHiiEx(-1, -1, NULL, STRING_TOKEN (STR_BCFG_WRITE_FAIL), gShellInstall1HiiHandle, Target == BcfgTargetBootOrder?(CHAR16*)L"BootOrder":(CHAR16*)L"DriverOrder", Status);\r
     return (SHELL_INVALID_PARAMETER);\r
   }\r
   return (SHELL_SUCCESS);\r
 }\r
 \r
+/**\r
+  Function to add optional data to an option.\r
+\r
+  @param[in] OptData      The optional data to add.\r
+  @param[in] CurrentOrder The pointer to the current order of items.\r
+  @param[in] OrderCount   The number if items in CurrentOrder.\r
+  @param[in] Target       The target of the operation.\r
+\r
+  @retval SHELL_SUCCESS   The operation was succesful.\r
+**/\r
 SHELL_STATUS\r
 EFIAPI\r
-BcfgDisplayDumpInstall(\r
+BcfgAddOptInstall1(\r
+  IN CONST CHAR16                 *OptData,\r
+  IN CONST UINT16                 *CurrentOrder,\r
+  IN CONST UINTN                  OrderCount,\r
+  IN CONST BCFG_OPERATION_TARGET  Target\r
+  )\r
+{\r
+  EFI_KEY_OPTION  NewKeyOption;\r
+  EFI_KEY_OPTION *KeyOptionBuffer;\r
+  SHELL_STATUS    ShellStatus;\r
+  EFI_STATUS      Status;\r
+  UINT16          OptionIndex;\r
+  UINT16          LoopCounter;\r
+  UINT64          Intermediate;\r
+  CONST CHAR16    *Temp;\r
+  CONST CHAR16    *Walker;\r
+  CHAR16          *FileName;\r
+  CHAR16          *Temp2;\r
+  CHAR16          *Data;\r
+  UINT16          KeyIndex;\r
+  CHAR16          VariableName[12];\r
+\r
+  SHELL_FILE_HANDLE FileHandle;\r
+\r
+  Status          = EFI_SUCCESS;\r
+  ShellStatus     = SHELL_SUCCESS;\r
+  Walker          = OptData;\r
+  FileName        = NULL;\r
+  Data            = NULL;\r
+  KeyOptionBuffer = NULL;\r
+\r
+  ZeroMem(&NewKeyOption, sizeof(EFI_KEY_OPTION));\r
+\r
+  while(Walker[0] == L' ') {\r
+    Walker++;\r
+  }\r
+\r
+  //\r
+  // Get the index of the variable we are changing.\r
+  //\r
+  Status = ShellConvertStringToUint64(Walker, &Intermediate, FALSE, TRUE);\r
+  if (EFI_ERROR(Status) || (((UINT16)Intermediate) != Intermediate) || StrStr(Walker, L" ") == NULL || ((UINT16)Intermediate) > ((UINT16)OrderCount)) {\r
+    ShellPrintHiiEx(-1, -1, NULL, STRING_TOKEN (STR_GEN_PROBLEM), gShellInstall1HiiHandle, L"Option Index");\r
+    ShellStatus = SHELL_INVALID_PARAMETER;\r
+    return (ShellStatus);\r
+  }\r
+  OptionIndex = (UINT16)Intermediate;\r
+\r
+  Temp = StrStr(Walker, L" ");\r
+  if (Temp != NULL) {\r
+    Walker = Temp;\r
+  }\r
+  while(Walker[0] == L' ') {\r
+    Walker++;\r
+  }\r
+\r
+  //\r
+  // determine whether we have file with data, quote delimited information, or a hot-key \r
+  //\r
+  if (Walker[0] == L'\"') {\r
+    //\r
+    // quoted filename or quoted information.\r
+    //\r
+    Temp = StrStr(Walker+1, L"\"");\r
+    if (Temp == NULL || StrLen(Temp) != 1) {\r
+      ShellPrintHiiEx(-1, -1, NULL, STRING_TOKEN (STR_GEN_PROBLEM), gShellInstall1HiiHandle, Walker);\r
+      ShellStatus = SHELL_INVALID_PARAMETER;\r
+    } else {\r
+      FileName = StrnCatGrow(&FileName, NULL, Walker+1, 0);\r
+      if (FileName == NULL) {\r
+        ShellPrintHiiEx(-1, -1, NULL, STRING_TOKEN (STR_GEN_OUT_MEM), gShellInstall1HiiHandle);\r
+        ShellStatus = SHELL_OUT_OF_RESOURCES;\r
+        return (ShellStatus);\r
+      }\r
+      Temp2 = StrStr(FileName, L"\"");\r
+      ASSERT(Temp2 != NULL);\r
+      Temp2[0] = CHAR_NULL;\r
+      Temp2++;\r
+      if (StrLen(Temp2)>0) {\r
+        ShellPrintHiiEx(-1, -1, NULL, STRING_TOKEN (STR_GEN_PROBLEM), gShellInstall1HiiHandle, Walker);\r
+        ShellStatus = SHELL_INVALID_PARAMETER;\r
+      }\r
+      if (EFI_ERROR(ShellFileExists(Walker))) {\r
+        //\r
+        // Not a file.  must be misc information.\r
+        //\r
+        Data     = FileName;\r
+        FileName = NULL;\r
+      } else {\r
+        FileName = StrnCatGrow(&FileName, NULL, Walker, 0);\r
+      }\r
+    }\r
+  } else {\r
+    //\r
+    // filename or hot key information.\r
+    //\r
+    if (StrStr(Walker, L" ") == NULL) {\r
+      //\r
+      // filename\r
+      //\r
+      if (EFI_ERROR(ShellFileExists(Walker))) {\r
+        ShellPrintHiiEx(-1, -1, NULL, STRING_TOKEN (STR_GEN_FIND_FAIL), gShellInstall1HiiHandle, Walker);\r
+        ShellStatus = SHELL_INVALID_PARAMETER;\r
+      } else {\r
+        FileName = StrnCatGrow(&FileName, NULL, Walker, 0);\r
+      }\r
+    } else {\r
+      if (Target != BcfgTargetBootOrder) {\r
+        ShellPrintHiiEx(-1, -1, NULL, STRING_TOKEN (STR_GEN_BOOT_ONLY), gShellInstall1HiiHandle);\r
+        ShellStatus = SHELL_INVALID_PARAMETER;\r
+      }\r
+\r
+      if (ShellStatus == SHELL_SUCCESS) {\r
+        //\r
+        // Get hot key information\r
+        //\r
+        Status = ShellConvertStringToUint64(Walker, &Intermediate, FALSE, TRUE);\r
+        if (EFI_ERROR(Status) || (((UINT32)Intermediate) != Intermediate) || StrStr(Walker, L" ") == NULL) {\r
+          ShellPrintHiiEx(-1, -1, NULL, STRING_TOKEN (STR_GEN_PROBLEM), gShellInstall1HiiHandle, Walker);\r
+          ShellStatus = SHELL_INVALID_PARAMETER;\r
+        }\r
+        NewKeyOption.KeyData = (UINT32)Intermediate;\r
+        Temp = StrStr(Walker, L" ");\r
+        if (Temp != NULL) {\r
+          Walker = Temp;\r
+        }\r
+        while(Walker[0] == L' ') {\r
+          Walker++;\r
+        }\r
+      }\r
+\r
+      if (ShellStatus == SHELL_SUCCESS) {\r
+        //\r
+        // Now we know how many EFI_INPUT_KEY structs we need to attach to the end of the EFI_KEY_OPTION struct.  \r
+        // Re-allocate with the added information.\r
+        //\r
+        KeyOptionBuffer = AllocateCopyPool(sizeof(EFI_KEY_OPTION) + (sizeof(EFI_INPUT_KEY) * KEY_OPTION_INPUT_KEY_COUNT (&NewKeyOption)), &NewKeyOption);\r
+        if (KeyOptionBuffer == NULL) {\r
+          ShellPrintHiiEx(-1, -1, NULL, STRING_TOKEN (STR_GEN_NO_MEM), gShellInstall1HiiHandle);\r
+          ShellStatus = SHELL_OUT_OF_RESOURCES;\r
+        }\r
+      }\r
+      for (LoopCounter = 0 ; ShellStatus == SHELL_SUCCESS && LoopCounter < KEY_OPTION_INPUT_KEY_COUNT (&NewKeyOption); LoopCounter++) {\r
+        //\r
+        // ScanCode\r
+        //\r
+        Status = ShellConvertStringToUint64(Walker, &Intermediate, FALSE, TRUE);\r
+        if (EFI_ERROR(Status) || (((UINT16)Intermediate) != Intermediate) || StrStr(Walker, L" ") == NULL) {\r
+          ShellPrintHiiEx(-1, -1, NULL, STRING_TOKEN (STR_GEN_PROBLEM), gShellInstall1HiiHandle, Walker);\r
+          ShellStatus = SHELL_INVALID_PARAMETER;\r
+        }\r
+        ((EFI_INPUT_KEY*)(((UINT8*)KeyOptionBuffer) + sizeof(EFI_KEY_OPTION)))[LoopCounter].ScanCode = (UINT16)Intermediate;\r
+        Temp = StrStr(Walker, L" ");\r
+        if (Temp != NULL) {\r
+          Walker = Temp;\r
+        }\r
+        while(Walker[0] == L' ') {\r
+          Walker++;\r
+        }\r
+\r
+        //\r
+        // UnicodeChar\r
+        //\r
+        Status = ShellConvertStringToUint64(Walker, &Intermediate, FALSE, TRUE);\r
+        if (EFI_ERROR(Status) || (((UINT16)Intermediate) != Intermediate)) {\r
+          ShellPrintHiiEx(-1, -1, NULL, STRING_TOKEN (STR_GEN_PROBLEM), gShellInstall1HiiHandle, Walker);\r
+          ShellStatus = SHELL_INVALID_PARAMETER;\r
+        }\r
+        ((EFI_INPUT_KEY*)(((UINT8*)KeyOptionBuffer) + sizeof(EFI_KEY_OPTION)))[LoopCounter].UnicodeChar = (UINT16)Intermediate;\r
+        Temp = StrStr(Walker, L" ");\r
+        if (Temp != NULL) {\r
+          Walker = Temp;\r
+        }\r
+        while(Walker[0] == L' ') {\r
+          Walker++;\r
+        }\r
+      }\r
+\r
+      if (ShellStatus == SHELL_SUCCESS) {\r
+        //\r
+        // Now do the BootOption / BootOptionCrc\r
+        //\r
+        ASSERT (OptionIndex <= OrderCount);\r
+        KeyOptionBuffer->BootOption    = CurrentOrder[OptionIndex];\r
+        Status = GetBootOptionCrc(&(KeyOptionBuffer->BootOptionCrc), KeyOptionBuffer->BootOption);\r
+        if (EFI_ERROR(Status)) {\r
+          ShellPrintHiiEx(-1, -1, NULL, STRING_TOKEN (STR_GEN_PROBLEM), gShellInstall1HiiHandle, L"Option Index");\r
+          ShellStatus = SHELL_INVALID_PARAMETER;\r
+        }        \r
+      }\r
+\r
+      if (ShellStatus == SHELL_SUCCESS) {\r
+        for (Temp2 = NULL, KeyIndex = 0 ; KeyIndex < 0xFFFF ; KeyIndex++) {\r
+          UnicodeSPrint(VariableName, sizeof(VariableName), L"Key%04x", KeyIndex);\r
+          Status = gRT->GetVariable(\r
+              VariableName,\r
+              (EFI_GUID*)&gEfiGlobalVariableGuid,\r
+              NULL,\r
+              (UINTN*)&Intermediate,\r
+              NULL);\r
+          if (Status == EFI_NOT_FOUND) {\r
+            break;\r
+          }\r
+        }\r
+        Status = gRT->SetVariable(\r
+          VariableName,\r
+          (EFI_GUID*)&gEfiGlobalVariableGuid,\r
+          EFI_VARIABLE_NON_VOLATILE|EFI_VARIABLE_BOOTSERVICE_ACCESS|EFI_VARIABLE_RUNTIME_ACCESS,\r
+          sizeof(EFI_KEY_OPTION) + (sizeof(EFI_INPUT_KEY) * KEY_OPTION_INPUT_KEY_COUNT (&NewKeyOption)),\r
+          KeyOptionBuffer);\r
+        if (EFI_ERROR(Status)) {\r
+          ShellPrintHiiEx(-1, -1, NULL, STRING_TOKEN (STR_BCFG_SET_VAR_FAIL), gShellInstall1HiiHandle, VariableName, Status);\r
+          ShellStatus = SHELL_INVALID_PARAMETER;\r
+        }   \r
+        ASSERT(FileName == NULL && Data == NULL);\r
+      }\r
+    }\r
+  }\r
+\r
+  //\r
+  // Shouldn't be possible to have have both. Neither is ok though.\r
+  //\r
+  ASSERT(FileName == NULL || Data == NULL);\r
+\r
+  if (ShellStatus == SHELL_SUCCESS && (FileName != NULL || Data != NULL)) {\r
+    if (FileName != NULL) {\r
+      //\r
+      // Open the file and populate the data buffer.\r
+      //\r
+      Status = ShellOpenFileByName(\r
+        FileName,\r
+        &FileHandle,\r
+        EFI_FILE_MODE_READ,\r
+        0);\r
+      if (!EFI_ERROR(Status)) {\r
+        Status = ShellGetFileSize(FileHandle, &Intermediate);\r
+      }\r
+      Data = AllocateZeroPool((UINTN)Intermediate);\r
+      if (Data == NULL) {\r
+        ShellPrintHiiEx(-1, -1, NULL, STRING_TOKEN (STR_GEN_NO_MEM), gShellInstall1HiiHandle);\r
+        ShellStatus = SHELL_OUT_OF_RESOURCES;\r
+      }\r
+      if (!EFI_ERROR(Status)) {\r
+        Status = ShellReadFile(FileHandle, (UINTN *)&Intermediate, Data);\r
+      }\r
+    } else {\r
+      Intermediate = StrSize(Data);\r
+    }\r
+\r
+    if (!EFI_ERROR(Status) && ShellStatus == SHELL_SUCCESS && Data != NULL) {\r
+      Status = UpdateOptionalData(CurrentOrder[OptionIndex], (UINTN)Intermediate, (UINT8*)Data, Target);\r
+      if (EFI_ERROR(Status)) {\r
+        ShellPrintHiiEx(-1, -1, NULL, STRING_TOKEN (STR_BCFG_SET_VAR_FAIL), gShellInstall1HiiHandle, VariableName, Status);\r
+        ShellStatus = SHELL_INVALID_PARAMETER;\r
+      }   \r
+    }\r
+    if (EFI_ERROR(Status) && ShellStatus == SHELL_SUCCESS) {\r
+      ShellPrintHiiEx(-1, -1, NULL, STRING_TOKEN (STR_BCFG_SET_VAR_FAIL), gShellInstall1HiiHandle, VariableName, Status);\r
+      ShellStatus = SHELL_INVALID_PARAMETER;\r
+    }   \r
+  }\r
+\r
+  SHELL_FREE_NON_NULL(Data);\r
+  SHELL_FREE_NON_NULL(KeyOptionBuffer);\r
+  SHELL_FREE_NON_NULL(FileName);\r
+  return ShellStatus;\r
+}\r
+\r
+/**\r
+  Function to dump the Bcfg information.\r
+\r
+  @param[in] Op             The operation.\r
+  @param[in] OrderCount     How many to dump.\r
+  @param[in] CurrentOrder   The pointer to the current order of items.\r
+  @param[in] VerboseOutput  TRUE for extra output.  FALSE otherwise.\r
+\r
+  @retval SHELL_SUCCESS           The dump was successful.\r
+  @retval SHELL_INVALID_PARAMETER A parameter was invalid.\r
+**/\r
+SHELL_STATUS\r
+EFIAPI\r
+BcfgDisplayDumpInstall1(\r
   IN CONST CHAR16   *Op,\r
-  IN CONST UINT16   *CurrentOrder,\r
   IN CONST UINTN    OrderCount,\r
+  IN CONST UINT16   *CurrentOrder,\r
   IN CONST BOOLEAN  VerboseOutput\r
   )\r
 {\r
@@ -433,6 +1017,11 @@ BcfgDisplayDumpInstall(
   CHAR16      *DevPathString;\r
   VOID        *DevPath;\r
 \r
+  if (OrderCount == 0) {\r
+    ShellPrintHiiEx(-1, -1, NULL, STRING_TOKEN(STR_BCFG_NONE), gShellInstall1HiiHandle);\r
+    return (SHELL_SUCCESS);\r
+  }\r
+\r
   for (LoopVar = 0 ; LoopVar < OrderCount ; LoopVar++) {\r
     Buffer      = NULL;\r
     BufferSize  = 0;\r
@@ -445,7 +1034,7 @@ BcfgDisplayDumpInstall(
         &BufferSize,\r
         Buffer);\r
     if (Status == EFI_BUFFER_TOO_SMALL) {\r
-      Buffer = AllocatePool(BufferSize);\r
+      Buffer = AllocateZeroPool(BufferSize);\r
       Status = gRT->GetVariable(\r
           VariableName,\r
           (EFI_GUID*)&gEfiGlobalVariableGuid,\r
@@ -455,30 +1044,43 @@ BcfgDisplayDumpInstall(
     }\r
 \r
     if (EFI_ERROR(Status) || Buffer == NULL) {\r
-      SHELL_FREE_NON_NULL(Buffer);\r
       ShellPrintHiiEx(-1, -1, NULL, STRING_TOKEN (STR_BCFG_READ_FAIL), gShellInstall1HiiHandle, VariableName, Status);\r
       return (SHELL_INVALID_PARAMETER);\r
     }\r
 \r
-    DevPath = AllocatePool(*(UINT16*)(Buffer+4));\r
-    CopyMem(DevPath, Buffer+6+StrSize((CHAR16*)(Buffer+6)), *(UINT16*)(Buffer+4));\r
-    DevPathString = gDevPathToText->ConvertDevicePathToText(DevPath, TRUE, FALSE);\r
+    if ((*(UINT16*)(Buffer+4)) != 0) {\r
+      DevPath = AllocateZeroPool(*(UINT16*)(Buffer+4));\r
+      CopyMem(DevPath, Buffer+6+StrSize((CHAR16*)(Buffer+6)), *(UINT16*)(Buffer+4));\r
+      DevPathString = ConvertDevicePathToText(DevPath, TRUE, FALSE);\r
+    } else {\r
+      DevPath       = NULL;\r
+      DevPathString = NULL;\r
+    }\r
     ShellPrintHiiEx(\r
       -1,\r
       -1,\r
       NULL,\r
       STRING_TOKEN(STR_BCFG_LOAD_OPTIONS),\r
       gShellInstall1HiiHandle,\r
-      Op,\r
       LoopVar,\r
+      VariableName,\r
       (CHAR16*)(Buffer+6),\r
       DevPathString,\r
       (StrSize((CHAR16*)(Buffer+6)) + *(UINT16*)(Buffer+4) + 6) <= BufferSize?L'N':L'Y');\r
     if (VerboseOutput) {\r
       for (LoopVar2 = (StrSize((CHAR16*)(Buffer+6)) + *(UINT16*)(Buffer+4) + 6);LoopVar2<BufferSize;LoopVar2++){\r
-        ShellPrintEx(-1, -1, L"%02x", Buffer[LoopVar2]);\r
+        ShellPrintEx(\r
+          -1,\r
+          -1,\r
+          NULL,\r
+          L"%02x",\r
+          Buffer[LoopVar2]);\r
       }\r
-      ShellPrintEx(-1, -1, L"\r\n");\r
+      ShellPrintEx(\r
+        -1,\r
+        -1,\r
+        NULL,\r
+        L"\r\n");\r
     }\r
 \r
     if (Buffer != NULL) {\r
@@ -494,15 +1096,20 @@ BcfgDisplayDumpInstall(
   return (SHELL_SUCCESS);\r
 }\r
 \r
+/**\r
+  Function to initialize the BCFG operation structure.\r
+\r
+  @param[in] Struct   The stuct to initialize.\r
+**/\r
 VOID\r
 EFIAPI\r
-InitBcfgStructInstall(\r
+InitBcfgStructInstall1(\r
   IN BGFG_OPERATION *Struct\r
   )\r
 {\r
   ASSERT(Struct != NULL);\r
-  Struct->Target      = BCFG_TARGET_MAX;\r
-  Struct->Type        = BCFG_TYPE_MAX;\r
+  Struct->Target      = BcfgTargetMax;\r
+  Struct->Type        = BcfgTypeMax;\r
   Struct->Number1     = 0;\r
   Struct->Number2     = 0;\r
   Struct->HandleIndex = 0;\r
@@ -540,13 +1147,14 @@ ShellCommandRunBcfgInstall (
   CONST CHAR16          *CurrentParam;\r
   BGFG_OPERATION        CurrentOperation;\r
   UINTN                 Length;\r
+  UINT64                Intermediate;\r
 \r
   Length              = 0;\r
   ProblemParam        = NULL;\r
   Package             = NULL;\r
   ShellStatus         = SHELL_SUCCESS;\r
 \r
-  InitBcfgStructInstall(&CurrentOperation);\r
+  InitBcfgStructInstall1(&CurrentOperation);\r
 \r
   //\r
   // initialize the shell lib (we must be in non-auto-init...)\r
@@ -570,47 +1178,51 @@ ShellCommandRunBcfgInstall (
       ASSERT(FALSE);\r
     }\r
   } else {\r
+    //\r
+    // Read in if we are doing -OPT\r
+    //\r
+    if (ShellCommandLineGetFlag(Package, L"-opt")) {\r
+      CurrentOperation.OptData = ShellCommandLineGetValue(Package, L"-opt");\r
+      if (CurrentOperation.OptData == NULL) {\r
+        ShellPrintHiiEx(-1, -1, NULL, STRING_TOKEN (STR_GEN_PROBLEM_VAL), gShellInstall1HiiHandle, L"-opt");\r
+        ShellStatus = SHELL_INVALID_PARAMETER;\r
+      }\r
+      CurrentOperation.Type = BcfgTypeOpt;\r
+    }\r
+\r
     //\r
     // small block to read the target of the operation\r
     //\r
-    if (ShellCommandLineGetFlag(Package, L"-opt") && ShellCommandLineGetCount(Package) < 2) {\r
-      ShellPrintHiiEx(-1, -1, NULL, STRING_TOKEN (STR_GEN_TOO_FEW), gShellInstall1HiiHandle);\r
-      ShellStatus = SHELL_INVALID_PARAMETER;\r
-    } else if (!ShellCommandLineGetFlag(Package, L"-opt") && ShellCommandLineGetCount(Package) < 3) {\r
+    if ((ShellCommandLineGetCount(Package) < 3 && CurrentOperation.Type != BcfgTypeOpt) ||\r
+        (ShellCommandLineGetCount(Package) < 2 && CurrentOperation.Type == BcfgTypeOpt)\r
+       ){\r
       ShellPrintHiiEx(-1, -1, NULL, STRING_TOKEN (STR_GEN_TOO_FEW), gShellInstall1HiiHandle);\r
       ShellStatus = SHELL_INVALID_PARAMETER;\r
     } else if (gUnicodeCollation->StriColl(gUnicodeCollation, (CHAR16*)ShellCommandLineGetRawValue(Package, 1), L"driver") == 0) {\r
-      CurrentOperation.Target = BCFG_TARGET_DRIVER_ORDER;\r
+      CurrentOperation.Target = BcfgTargetDriverOrder;\r
     } else if (gUnicodeCollation->StriColl(gUnicodeCollation, (CHAR16*)ShellCommandLineGetRawValue(Package, 1), L"boot") == 0) {\r
-      CurrentOperation.Target = BCFG_TARGET_BOOT_ORDER;\r
+      CurrentOperation.Target = BcfgTargetBootOrder;\r
     } else {\r
       ShellPrintHiiEx(-1, -1, NULL, STRING_TOKEN (STR_GEN_NO_DRIVER_BOOT), gShellInstall1HiiHandle);\r
       ShellStatus = SHELL_INVALID_PARAMETER;\r
     }\r
 \r
-    //\r
-    // Read in if we are doing -OPT\r
-    //\r
-    if (ShellStatus == SHELL_SUCCESS && CurrentOperation.Target < BCFG_TARGET_MAX && ShellCommandLineGetFlag(Package, L"-opt")) {\r
-      CurrentOperation.OptData = ShellCommandLineGetValue(Package, L"-opt");\r
-      CurrentOperation.Type = BCFG_TYPE_OPT;\r
-    }\r
 \r
     //\r
     // Read in the boot or driver order environment variable (not needed for opt)\r
     //\r
-    if (ShellStatus == SHELL_SUCCESS && CurrentOperation.Target < BCFG_TARGET_MAX && CurrentOperation.Type != BCFG_TYPE_OPT) {\r
+    if (ShellStatus == SHELL_SUCCESS && CurrentOperation.Target < BcfgTargetMax) {\r
       Length = 0;\r
       Status = gRT->GetVariable(\r
-        CurrentOperation.Target == BCFG_TARGET_BOOT_ORDER?(CHAR16*)L"BootOrder":(CHAR16*)L"DriverOrder",\r
+        CurrentOperation.Target == BcfgTargetBootOrder?(CHAR16*)L"BootOrder":(CHAR16*)L"DriverOrder",\r
         (EFI_GUID*)&gEfiGlobalVariableGuid,\r
         NULL,\r
         &Length,\r
         CurrentOperation.Order);\r
       if (Status == EFI_BUFFER_TOO_SMALL) {\r
-        CurrentOperation.Order = AllocatePool(Length+(4*sizeof(CurrentOperation.Order[0])));\r
+        CurrentOperation.Order = AllocateZeroPool(Length+(4*sizeof(CurrentOperation.Order[0])));\r
         Status = gRT->GetVariable(\r
-          CurrentOperation.Target == BCFG_TARGET_BOOT_ORDER?(CHAR16*)L"BootOrder":(CHAR16*)L"DriverOrder",\r
+          CurrentOperation.Target == BcfgTargetBootOrder?(CHAR16*)L"BootOrder":(CHAR16*)L"DriverOrder",\r
           (EFI_GUID*)&gEfiGlobalVariableGuid,\r
           NULL,\r
           &Length,\r
@@ -621,15 +1233,11 @@ ShellCommandRunBcfgInstall (
     //\r
     // large block to read the type of operation and verify parameter types for the info.\r
     //\r
-    if (ShellStatus == SHELL_SUCCESS && CurrentOperation.Target < BCFG_TARGET_MAX) {\r
+    if (ShellStatus == SHELL_SUCCESS && CurrentOperation.Target < BcfgTargetMax) {\r
       for (ParamNumber = 2 ; ParamNumber < ShellCommandLineGetCount(Package) && ShellStatus == SHELL_SUCCESS; ParamNumber++) {\r
         CurrentParam = ShellCommandLineGetRawValue(Package, ParamNumber);\r
         if        (gUnicodeCollation->StriColl(gUnicodeCollation, (CHAR16*)CurrentParam, L"dump") == 0)    {\r
-          if (ShellCommandLineGetCount(Package) > 3) {\r
-            ShellPrintHiiEx(-1, -1, NULL, STRING_TOKEN (STR_GEN_TOO_MANY), gShellInstall1HiiHandle);\r
-            ShellStatus = SHELL_INVALID_PARAMETER;\r
-          }\r
-          CurrentOperation.Type = BCFG_TYPE_DUMP;\r
+          CurrentOperation.Type = BcfgTypeDump;\r
         } else if (ShellCommandLineGetFlag(Package, L"-v")) {\r
           ShellPrintHiiEx(-1, -1, NULL, STRING_TOKEN (STR_GEN_PROBLEM), gShellInstall1HiiHandle, L"-v (without dump)");\r
           ShellStatus = SHELL_INVALID_PARAMETER;\r
@@ -638,13 +1246,14 @@ ShellCommandRunBcfgInstall (
             ShellPrintHiiEx(-1, -1, NULL, STRING_TOKEN (STR_GEN_TOO_FEW), gShellInstall1HiiHandle);\r
             ShellStatus = SHELL_INVALID_PARAMETER;\r
           }\r
-          CurrentOperation.Type = BCFG_TYPE_ADD;\r
+          CurrentOperation.Type = BcfgTypeAdd;\r
           CurrentParam = ShellCommandLineGetRawValue(Package, ++ParamNumber);\r
           if (CurrentParam == NULL || !ShellIsHexOrDecimalNumber(CurrentParam, TRUE, FALSE)) {\r
             ShellPrintHiiEx(-1, -1, NULL, STRING_TOKEN (STR_GEN_PROBLEM), gShellInstall1HiiHandle, CurrentParam);\r
             ShellStatus = SHELL_INVALID_PARAMETER;\r
           } else {\r
-            CurrentOperation.Number1     = (UINT16)StrHexToUintn(CurrentParam);\r
+            Status = ShellConvertStringToUint64(CurrentParam, &Intermediate, TRUE, FALSE);\r
+            CurrentOperation.Number1     = (UINT16)Intermediate;\r
             ASSERT(CurrentOperation.FileName == NULL);\r
             CurrentOperation.FileName    = StrnCatGrow(&CurrentOperation.FileName   , NULL, ShellCommandLineGetRawValue(Package, ++ParamNumber), 0);\r
             ASSERT(CurrentOperation.Description == NULL);\r
@@ -654,38 +1263,40 @@ ShellCommandRunBcfgInstall (
           if ((ParamNumber + 3) >= ShellCommandLineGetCount(Package)) {\r
             ShellPrintHiiEx(-1, -1, NULL, STRING_TOKEN (STR_GEN_TOO_FEW), gShellInstall1HiiHandle);\r
             ShellStatus = SHELL_INVALID_PARAMETER;\r
+          }\r
+          CurrentOperation.Type = BcfgTypeAddp;\r
+          CurrentParam = ShellCommandLineGetRawValue(Package, ++ParamNumber);\r
+          if (CurrentParam == NULL || !ShellIsHexOrDecimalNumber(CurrentParam, TRUE, FALSE)) {\r
+            ShellPrintHiiEx(-1, -1, NULL, STRING_TOKEN (STR_GEN_PROBLEM), gShellInstall1HiiHandle, CurrentParam);\r
+            ShellStatus = SHELL_INVALID_PARAMETER;\r
           } else {\r
-            CurrentOperation.Type = BCFG_TYPE_ADDP;\r
-            CurrentParam = ShellCommandLineGetRawValue(Package, ++ParamNumber);\r
-            if (CurrentParam == NULL || !ShellIsHexOrDecimalNumber(CurrentParam, TRUE, FALSE)) {\r
-              ShellPrintHiiEx(-1, -1, NULL, STRING_TOKEN (STR_GEN_PROBLEM), gShellInstall1HiiHandle, CurrentParam);\r
-              ShellStatus = SHELL_INVALID_PARAMETER;\r
-            } else {\r
-              CurrentOperation.Number1     = (UINT16)StrHexToUintn(CurrentParam);\r
-              ASSERT(CurrentOperation.FileName == NULL);\r
-              CurrentOperation.FileName    = StrnCatGrow(&CurrentOperation.FileName   , NULL, ShellCommandLineGetRawValue(Package, ++ParamNumber), 0);\r
-              ASSERT(CurrentOperation.Description == NULL);\r
-              CurrentOperation.Description = StrnCatGrow(&CurrentOperation.Description, NULL, ShellCommandLineGetRawValue(Package, ++ParamNumber), 0);\r
-            }\r
+            Status = ShellConvertStringToUint64(CurrentParam, &Intermediate, TRUE, FALSE);\r
+            CurrentOperation.Number1     = (UINT16)Intermediate;\r
+            ASSERT(CurrentOperation.FileName == NULL);\r
+            CurrentOperation.FileName    = StrnCatGrow(&CurrentOperation.FileName   , NULL, ShellCommandLineGetRawValue(Package, ++ParamNumber), 0);\r
+            ASSERT(CurrentOperation.Description == NULL);\r
+            CurrentOperation.Description = StrnCatGrow(&CurrentOperation.Description, NULL, ShellCommandLineGetRawValue(Package, ++ParamNumber), 0);\r
           }\r
         } else if (gUnicodeCollation->StriColl(gUnicodeCollation, (CHAR16*)CurrentParam, L"addh") == 0)    {\r
           if ((ParamNumber + 3) >= ShellCommandLineGetCount(Package)) {\r
             ShellPrintHiiEx(-1, -1, NULL, STRING_TOKEN (STR_GEN_TOO_FEW), gShellInstall1HiiHandle);\r
             ShellStatus = SHELL_INVALID_PARAMETER;\r
           }\r
-          CurrentOperation.Type = BCFG_TYPE_ADDH;\r
+          CurrentOperation.Type = BcfgTypeAddh;\r
           CurrentParam = ShellCommandLineGetRawValue(Package, ++ParamNumber);\r
           if (CurrentParam == NULL || !ShellIsHexOrDecimalNumber(CurrentParam, TRUE, FALSE)) {\r
             ShellPrintHiiEx(-1, -1, NULL, STRING_TOKEN (STR_GEN_PROBLEM), gShellInstall1HiiHandle, CurrentParam);\r
             ShellStatus = SHELL_INVALID_PARAMETER;\r
           } else {\r
-            CurrentOperation.Number1     = (UINT16)StrHexToUintn(CurrentParam);\r
+            Status = ShellConvertStringToUint64(CurrentParam, &Intermediate, TRUE, FALSE);\r
+            CurrentOperation.Number1     = (UINT16)Intermediate;\r
             CurrentParam = ShellCommandLineGetRawValue(Package, ++ParamNumber);\r
             if (CurrentParam == NULL || !ShellIsHexOrDecimalNumber(CurrentParam, TRUE, FALSE)) {\r
               ShellPrintHiiEx(-1, -1, NULL, STRING_TOKEN (STR_GEN_PROBLEM), gShellInstall1HiiHandle, CurrentParam);\r
               ShellStatus = SHELL_INVALID_PARAMETER;\r
             } else {\r
-              CurrentOperation.HandleIndex = (UINT16)StrHexToUintn(CurrentParam);\r
+              Status = ShellConvertStringToUint64(CurrentParam, &Intermediate, TRUE, FALSE);\r
+              CurrentOperation.HandleIndex = (UINT16)Intermediate;\r
               ASSERT(CurrentOperation.Description == NULL);\r
               CurrentOperation.Description = StrnCatGrow(&CurrentOperation.Description, NULL, ShellCommandLineGetRawValue(Package, ++ParamNumber), 0);\r
             }\r
@@ -695,15 +1306,16 @@ ShellCommandRunBcfgInstall (
             ShellPrintHiiEx(-1, -1, NULL, STRING_TOKEN (STR_GEN_TOO_FEW), gShellInstall1HiiHandle);\r
             ShellStatus = SHELL_INVALID_PARAMETER;\r
           }\r
-          CurrentOperation.Type = BCFG_TYPE_RM;\r
+          CurrentOperation.Type = BcfgTypeRm;\r
           CurrentParam = ShellCommandLineGetRawValue(Package, ++ParamNumber);\r
           if (CurrentParam == NULL || !ShellIsHexOrDecimalNumber(CurrentParam, TRUE, FALSE)) {\r
             ShellPrintHiiEx(-1, -1, NULL, STRING_TOKEN (STR_GEN_PROBLEM), gShellInstall1HiiHandle, CurrentParam);\r
             ShellStatus = SHELL_INVALID_PARAMETER;\r
           } else {\r
-            CurrentOperation.Number1     = (UINT16)StrHexToUintn(CurrentParam);\r
+            Status = ShellConvertStringToUint64(CurrentParam, &Intermediate, TRUE, FALSE);\r
+            CurrentOperation.Number1     = (UINT16)Intermediate;\r
             if (CurrentOperation.Number1 > (Length / sizeof(CurrentOperation.Order[0]))){\r
-              ShellPrintHiiEx(-1, -1, NULL, STRING_TOKEN (STR_BCFG_LOCATION_RANGE), gShellInstall1HiiHandle, Length / sizeof(CurrentOperation.Order[0]));\r
+              ShellPrintHiiEx(-1, -1, NULL, STRING_TOKEN (STR_BCFG_NUMB_RANGE), gShellInstall1HiiHandle, Length / sizeof(CurrentOperation.Order[0]));\r
               ShellStatus = SHELL_INVALID_PARAMETER;\r
             }\r
           }\r
@@ -712,15 +1324,16 @@ ShellCommandRunBcfgInstall (
             ShellPrintHiiEx(-1, -1, NULL, STRING_TOKEN (STR_GEN_TOO_FEW), gShellInstall1HiiHandle);\r
             ShellStatus = SHELL_INVALID_PARAMETER;\r
           }\r
-          CurrentOperation.Type = BCFG_TYPE_MV;\r
+          CurrentOperation.Type = BcfgTypeMv;\r
           CurrentParam = ShellCommandLineGetRawValue(Package, ++ParamNumber);\r
           if (CurrentParam == NULL || !ShellIsHexOrDecimalNumber(CurrentParam, TRUE, FALSE)) {\r
             ShellPrintHiiEx(-1, -1, NULL, STRING_TOKEN (STR_GEN_PROBLEM), gShellInstall1HiiHandle, CurrentParam);\r
             ShellStatus = SHELL_INVALID_PARAMETER;\r
           } else {\r
-            CurrentOperation.Number1     = (UINT16)StrHexToUintn(CurrentParam);\r
+            Status = ShellConvertStringToUint64(CurrentParam, &Intermediate, TRUE, FALSE);\r
+            CurrentOperation.Number1     = (UINT16)Intermediate;\r
             if (CurrentOperation.Number1 > (Length / sizeof(CurrentOperation.Order[0]))){\r
-              ShellPrintHiiEx(-1, -1, NULL, STRING_TOKEN (STR_BCFG_LOCATION_RANGE), gShellInstall1HiiHandle, Length / sizeof(CurrentOperation.Order[0]));\r
+              ShellPrintHiiEx(-1, -1, NULL, STRING_TOKEN (STR_BCFG_NUMB_RANGE), gShellInstall1HiiHandle, Length / sizeof(CurrentOperation.Order[0]));\r
               ShellStatus = SHELL_INVALID_PARAMETER;\r
             }\r
             CurrentParam = ShellCommandLineGetRawValue(Package, ++ParamNumber);\r
@@ -728,13 +1341,14 @@ ShellCommandRunBcfgInstall (
               ShellPrintHiiEx(-1, -1, NULL, STRING_TOKEN (STR_GEN_PROBLEM), gShellInstall1HiiHandle, CurrentParam);\r
               ShellStatus = SHELL_INVALID_PARAMETER;\r
             } else {\r
-              CurrentOperation.Number2     = (UINT16)StrHexToUintn(CurrentParam);\r
+              Status = ShellConvertStringToUint64(CurrentParam, &Intermediate, TRUE, FALSE);\r
+              CurrentOperation.Number2     = (UINT16)Intermediate;\r
             }\r
             if (CurrentOperation.Number2 == CurrentOperation.Number1\r
               ||CurrentOperation.Number1 > (Length / sizeof(CurrentOperation.Order[0]))\r
               ||CurrentOperation.Number2 > (Length / sizeof(CurrentOperation.Order[0]))\r
              ){\r
-              ShellPrintHiiEx(-1, -1, NULL, STRING_TOKEN (STR_BCFG_LOCATION_RANGE), gShellInstall1HiiHandle, Length / sizeof(CurrentOperation.Order[0]));\r
+              ShellPrintHiiEx(-1, -1, NULL, STRING_TOKEN (STR_BCFG_NUMB_RANGE), gShellInstall1HiiHandle, Length / sizeof(CurrentOperation.Order[0]));\r
               ShellStatus = SHELL_INVALID_PARAMETER;\r
             }\r
           }\r
@@ -744,51 +1358,53 @@ ShellCommandRunBcfgInstall (
         }\r
       }\r
     }\r
-    if (ShellStatus == SHELL_SUCCESS && CurrentOperation.Target < BCFG_TARGET_MAX && CurrentOperation.Type < BCFG_TYPE_MAX) {\r
+    if (ShellStatus == SHELL_SUCCESS && CurrentOperation.Target < BcfgTargetMax && CurrentOperation.Type < BcfgTypeMax) {\r
       //\r
       // we have all the info.  Do the work\r
       //\r
       switch (CurrentOperation.Type) {\r
-        case   BCFG_TYPE_DUMP:\r
-          ShellStatus = BcfgDisplayDumpInstall(\r
-            CurrentOperation.Target == BCFG_TARGET_BOOT_ORDER?L"Boot":L"Driver",\r
-            CurrentOperation.Order,\r
+        case   BcfgTypeDump:\r
+          ShellStatus = BcfgDisplayDumpInstall1(\r
+            CurrentOperation.Target == BcfgTargetBootOrder?L"Boot":L"Driver",\r
             Length / sizeof(CurrentOperation.Order[0]),\r
+            CurrentOperation.Order,\r
             ShellCommandLineGetFlag(Package, L"-v"));\r
           break;\r
-        case   BCFG_TYPE_MV:\r
-          ShellStatus = BcfgMoveInstall(\r
+        case   BcfgTypeMv:\r
+          ShellStatus = BcfgMoveInstall1(\r
             CurrentOperation.Target,\r
             CurrentOperation.Order,\r
             Length / sizeof(CurrentOperation.Order[0]),\r
             CurrentOperation.Number1,\r
             CurrentOperation.Number2);\r
           break;\r
-        case   BCFG_TYPE_RM:\r
-          ShellStatus = BcfgRemoveInstall(\r
+        case   BcfgTypeRm:\r
+          ShellStatus = BcfgRemoveInstall1(\r
             CurrentOperation.Target,\r
             CurrentOperation.Order,\r
-            (Length / sizeof(CurrentOperation.Order[0])),\r
+            Length / sizeof(CurrentOperation.Order[0]),\r
             CurrentOperation.Number1);\r
           break;\r
-        case   BCFG_TYPE_ADD:\r
-        case   BCFG_TYPE_ADDP:\r
-        case   BCFG_TYPE_ADDH:\r
-          ShellStatus = BcfgAddInstall(\r
+        case   BcfgTypeAdd:\r
+        case   BcfgTypeAddp:\r
+        case   BcfgTypeAddh:\r
+          ShellStatus = BcfgAddInstall1(\r
             CurrentOperation.Number1,\r
             CurrentOperation.FileName,\r
-            CurrentOperation.Description,\r
+            CurrentOperation.Description==NULL?L"":CurrentOperation.Description,\r
             CurrentOperation.Order,\r
             Length / sizeof(CurrentOperation.Order[0]),\r
             CurrentOperation.Target,\r
-            (BOOLEAN)(CurrentOperation.Type == BCFG_TYPE_ADDH),\r
-            (BOOLEAN)(CurrentOperation.Type == BCFG_TYPE_ADD ),\r
+            (BOOLEAN)(CurrentOperation.Type == BcfgTypeAddh),\r
+            (BOOLEAN)(CurrentOperation.Type == BcfgTypeAddp),\r
             CurrentOperation.HandleIndex);\r
           break;\r
-        case   BCFG_TYPE_OPT:\r
-          ShellStatus = BcfgAddOptInstall(\r
-            CurrentOperation.Target,\r
-            CurrentOperation.OptData);\r
+        case   BcfgTypeOpt:\r
+          ShellStatus = BcfgAddOptInstall1(\r
+            CurrentOperation.OptData,\r
+            CurrentOperation.Order,\r
+            Length / sizeof(CurrentOperation.Order[0]),\r
+            CurrentOperation.Target);\r
           break;\r
         default:\r
           ASSERT(FALSE);\r