]> git.proxmox.com Git - mirror_edk2.git/blobdiff - MdeModulePkg/Library/UefiHiiLib/HiiLib.c
MdeModulePkg: Refine casting expression result to bigger size
[mirror_edk2.git] / MdeModulePkg / Library / UefiHiiLib / HiiLib.c
index 02887bf76c7bb155d0bf944d641ba3e3dbd93913..a2abf26980b9ddaec07cdced210fbe68dd3b93b0 100644 (file)
@@ -1,7 +1,7 @@
 /** @file\r
   HII Library implementation that uses DXE protocols and services.\r
 \r
-  Copyright (c) 2006 - 2013, Intel Corporation. All rights reserved.<BR>\r
+  Copyright (c) 2006 - 2017, 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
@@ -31,6 +31,11 @@ typedef struct {
   UINT8               Scope;\r
 } IFR_BLOCK_DATA;\r
 \r
+typedef struct {\r
+  EFI_VARSTORE_ID     VarStoreId;\r
+  UINT16              Size;\r
+} IFR_VARSTORAGE_DATA;\r
+\r
 //\r
 // <ConfigHdr> Template\r
 //\r
@@ -372,6 +377,127 @@ HiiGetHiiHandles (
   }\r
 }\r
 \r
+/**\r
+  This function allows a caller to extract the form set opcode form the Hii Handle.\r
+  The returned buffer is allocated using AllocatePool().The caller is responsible \r
+  for freeing the allocated buffer using FreePool().\r
+\r
+  @param Handle            The HII handle.\r
+  @param Buffer            On return, points to a pointer which point to the buffer that contain the formset opcode.\r
+  @param BufferSize        On return, points to the length of the buffer.\r
+\r
+  @retval EFI_OUT_OF_RESOURCES   No enough memory resource is allocated.\r
+  @retval EFI_NOT_FOUND          Can't find the package data for the input Handle.\r
+  @retval EFI_INVALID_PARAMETER  The input parameters are not correct.\r
+  @retval EFI_SUCCESS            Get the formset opcode from the hii handle successfully.\r
+\r
+**/\r
+EFI_STATUS\r
+EFIAPI\r
+HiiGetFormSetFromHiiHandle(\r
+  IN  EFI_HII_HANDLE     Handle,\r
+  OUT EFI_IFR_FORM_SET   **Buffer,\r
+  OUT UINTN              *BufferSize\r
+  )\r
+{\r
+  EFI_STATUS                   Status;\r
+  UINTN                        PackageListSize;\r
+  UINTN                        TempSize;\r
+  EFI_HII_PACKAGE_LIST_HEADER  *HiiPackageList;\r
+  UINT8                        *Package;\r
+  UINT8                        *OpCodeData;\r
+  UINT8                        *FormSetBuffer;\r
+  UINT8                        *TempBuffer;\r
+  UINT32                       Offset;\r
+  UINT32                       Offset2;\r
+  UINT32                       PackageListLength;\r
+  EFI_HII_PACKAGE_HEADER       PackageHeader;\r
+\r
+  TempSize = 0;\r
+  FormSetBuffer = NULL;\r
+  TempBuffer    = NULL;\r
+\r
+  //\r
+  // Get HII PackageList\r
+  //\r
+  PackageListSize = 0;\r
+  HiiPackageList = NULL;\r
+  Status = gHiiDatabase->ExportPackageLists (gHiiDatabase, Handle, &PackageListSize, HiiPackageList);\r
+  if (EFI_ERROR (Status) && (Status != EFI_BUFFER_TOO_SMALL)) {\r
+    return Status;\r
+  }\r
+\r
+  HiiPackageList = AllocatePool (PackageListSize);\r
+  if (HiiPackageList == NULL) {\r
+    return EFI_OUT_OF_RESOURCES;\r
+  }\r
+\r
+  Status = gHiiDatabase->ExportPackageLists (gHiiDatabase, Handle, &PackageListSize, HiiPackageList);\r
+  ASSERT_EFI_ERROR (Status);\r
+\r
+  //\r
+  // Get Form package from this HII package List\r
+  //\r
+  Status = EFI_NOT_FOUND;\r
+  Offset = sizeof (EFI_HII_PACKAGE_LIST_HEADER);\r
+  PackageListLength = ReadUnaligned32 (&HiiPackageList->PackageLength);\r
+\r
+  while (Offset < PackageListLength) {\r
+    Package = ((UINT8 *) HiiPackageList) + Offset;\r
+    CopyMem (&PackageHeader, Package, sizeof (EFI_HII_PACKAGE_HEADER));\r
+    Offset += PackageHeader.Length;\r
+\r
+    if (PackageHeader.Type != EFI_HII_PACKAGE_FORMS) {\r
+      continue;\r
+    }\r
+\r
+    //\r
+    // Search FormSet Opcode in this Form Package\r
+    //\r
+    Offset2 = sizeof (EFI_HII_PACKAGE_HEADER);\r
+    while (Offset2 < PackageHeader.Length) {\r
+      OpCodeData = Package + Offset2;\r
+      Offset2 += ((EFI_IFR_OP_HEADER *) OpCodeData)->Length;\r
+\r
+      if (((EFI_IFR_OP_HEADER *) OpCodeData)->OpCode != EFI_IFR_FORM_SET_OP) {\r
+        continue;\r
+      }\r
+\r
+      if (FormSetBuffer != NULL){\r
+        TempBuffer = AllocateCopyPool (TempSize + ((EFI_IFR_OP_HEADER *) OpCodeData)->Length, FormSetBuffer);\r
+        FreePool(FormSetBuffer);\r
+        FormSetBuffer = NULL;\r
+        if (TempBuffer == NULL) {\r
+          Status = EFI_OUT_OF_RESOURCES;\r
+          goto Done;\r
+        }\r
+        CopyMem (TempBuffer + TempSize,  OpCodeData, ((EFI_IFR_OP_HEADER *) OpCodeData)->Length);\r
+      } else {\r
+        TempBuffer = AllocateCopyPool (TempSize + ((EFI_IFR_OP_HEADER *) OpCodeData)->Length, OpCodeData);\r
+        if (TempBuffer == NULL) {\r
+          Status = EFI_OUT_OF_RESOURCES;\r
+          goto Done;\r
+        }\r
+      }\r
+      TempSize += ((EFI_IFR_OP_HEADER *) OpCodeData)->Length;\r
+      FormSetBuffer = TempBuffer;\r
+\r
+      Status = EFI_SUCCESS;\r
+      //\r
+      //One form package has one formset, exit current form package to search other form package in the packagelist.\r
+      //\r
+      break;\r
+    }\r
+  }\r
+Done:\r
+  FreePool (HiiPackageList);\r
+\r
+  *BufferSize = TempSize;\r
+  *Buffer = (EFI_IFR_FORM_SET *)FormSetBuffer;\r
+\r
+  return Status;\r
+}\r
+\r
 /**\r
   Converts all hex dtring characters in range ['A'..'F'] to ['a'..'f'] for \r
   hex digits that appear between a '=' and a '&' in a config string.\r
@@ -570,17 +696,17 @@ InternalHiiBrowserCallback (
 \r
   @param[in]  Guid          Pointer to an EFI_GUID that is the routing information\r
                             GUID.  Each of the 16 bytes in Guid is converted to \r
-                            a 2 Unicode character hexidecimal string.  This is \r
+                            a 2 Unicode character hexadecimal string.  This is\r
                             an optional parameter that may be NULL.\r
   @param[in]  Name          Pointer to a Null-terminated Unicode string that is \r
                             the routing information NAME.  This is an optional \r
                             parameter that may be NULL.  Each 16-bit Unicode \r
                             character in Name is converted to a 4 character Unicode \r
-                            hexidecimal string.                        \r
+                            hexadecimal string.\r
   @param[in]  DriverHandle  The driver handle which supports a Device Path Protocol\r
                             that is the routing information PATH.  Each byte of\r
                             the Device Path associated with DriverHandle is converted\r
-                            to a 2 Unicode character hexidecimal string.\r
+                            to a 2 Unicode character hexadecimal string.\r
 \r
   @retval NULL   DriverHandle does not support the Device Path Protocol.\r
   @retval Other  A pointer to the Null-terminate Unicode <ConfigHdr> string\r
@@ -601,6 +727,7 @@ HiiConstructConfigHdr (
   CHAR16                    *ReturnString;\r
   UINTN                     Index;\r
   UINT8                     *Buffer;\r
+  UINTN                     MaxLen;\r
 \r
   //\r
   // Compute the length of Name in Unicode characters.  \r
@@ -631,7 +758,8 @@ HiiConstructConfigHdr (
   // GUID=<HexCh>32&NAME=<Char>NameLength&PATH=<HexChar>DevicePathSize <Null>\r
   // | 5 | sizeof (EFI_GUID) * 2 | 6 | NameStrLen*4 | 6 | DevicePathSize * 2 | 1 |\r
   //\r
-  String = AllocateZeroPool ((5 + sizeof (EFI_GUID) * 2 + 6 + NameLength * 4 + 6 + DevicePathSize * 2 + 1) * sizeof (CHAR16));\r
+  MaxLen = 5 + sizeof (EFI_GUID) * 2 + 6 + NameLength * 4 + 6 + DevicePathSize * 2 + 1;\r
+  String = AllocateZeroPool (MaxLen * sizeof (CHAR16));\r
   if (String == NULL) {\r
     return NULL;\r
   }\r
@@ -639,7 +767,8 @@ HiiConstructConfigHdr (
   //\r
   // Start with L"GUID="\r
   //\r
-  ReturnString = StrCpy (String, L"GUID=");\r
+  StrCpyS (String, MaxLen, L"GUID=");\r
+  ReturnString = String;\r
   String += StrLen (String);\r
 \r
   if (Guid != NULL) {\r
@@ -647,14 +776,21 @@ HiiConstructConfigHdr (
     // Append Guid converted to <HexCh>32\r
     //\r
     for (Index = 0, Buffer = (UINT8 *)Guid; Index < sizeof (EFI_GUID); Index++) {\r
-      String += UnicodeValueToString (String, PREFIX_ZERO | RADIX_HEX, *(Buffer++), 2);\r
+      UnicodeValueToStringS (\r
+        String,\r
+        MaxLen * sizeof (CHAR16) - ((UINTN)String - (UINTN)ReturnString),\r
+        PREFIX_ZERO | RADIX_HEX,\r
+        *(Buffer++),\r
+        2\r
+        );\r
+      String += StrnLenS (String, MaxLen - ((UINTN)String - (UINTN)ReturnString) / sizeof (CHAR16));\r
     }\r
   }\r
   \r
   //\r
   // Append L"&NAME="\r
   //\r
-  StrCpy (String, L"&NAME=");\r
+  StrCatS (ReturnString, MaxLen, L"&NAME=");\r
   String += StrLen (String);\r
 \r
   if (Name != NULL) {\r
@@ -662,21 +798,35 @@ HiiConstructConfigHdr (
     // Append Name converted to <Char>NameLength\r
     //\r
     for (; *Name != L'\0'; Name++) {\r
-      String += UnicodeValueToString (String, PREFIX_ZERO | RADIX_HEX, *Name, 4);\r
+      UnicodeValueToStringS (\r
+        String,\r
+        sizeof (CHAR16) * MaxLen - ((UINTN)String - (UINTN)ReturnString),\r
+        PREFIX_ZERO | RADIX_HEX,\r
+        *Name,\r
+        4\r
+        );\r
+      String += StrnLenS (String, MaxLen - ((UINTN)String - (UINTN)ReturnString) / sizeof (CHAR16));\r
     }\r
   }\r
 \r
   //\r
   // Append L"&PATH="\r
   //\r
-  StrCpy (String, L"&PATH=");\r
+  StrCatS (ReturnString, MaxLen, L"&PATH=");\r
   String += StrLen (String);\r
 \r
   //\r
   // Append the device path associated with DriverHandle converted to <HexChar>DevicePathSize\r
   //\r
   for (Index = 0, Buffer = (UINT8 *)DevicePath; Index < DevicePathSize; Index++) {\r
-    String += UnicodeValueToString (String, PREFIX_ZERO | RADIX_HEX, *(Buffer++), 2);\r
+    UnicodeValueToStringS (\r
+      String,\r
+      sizeof (CHAR16) * MaxLen - ((UINTN)String - (UINTN)ReturnString),\r
+      PREFIX_ZERO | RADIX_HEX,\r
+      *(Buffer++),\r
+      2\r
+      );\r
+    String += StrnLenS (String, MaxLen - ((UINTN)String - (UINTN)ReturnString) / sizeof (CHAR16));\r
   }\r
 \r
   //\r
@@ -781,7 +931,7 @@ InternalHiiGetBufferFromString (
     StringPtr = (CHAR16 *) DataBuffer;\r
     ZeroMem (TemStr, sizeof (TemStr));\r
     for (Index = 0; Index < Length; Index += 4) {\r
-      StrnCpy (TemStr, ConfigHdr + Index, 4);\r
+      StrnCpyS (TemStr, sizeof (TemStr) / sizeof (CHAR16), ConfigHdr + Index, 4);\r
       StringPtr[Index/4] = (CHAR16) StrHexToUint64 (TemStr);\r
     }\r
     //\r
@@ -913,49 +1063,103 @@ InternalHiiGetValueOfNumber (
   return EFI_SUCCESS;\r
 }\r
 \r
+/**\r
+  Get value from config request resp string.\r
+\r
+  @param ConfigElement           ConfigResp string contains the current setting.\r
+  @param VarName                 The variable name which need to get value.\r
+  @param VarValue                The return value.\r
+  \r
+  @retval EFI_SUCCESS            Get the value for the VarName\r
+  @retval EFI_OUT_OF_RESOURCES   The memory is not enough.\r
+**/\r
+EFI_STATUS\r
+GetValueFromRequest (\r
+  IN CHAR16                       *ConfigElement,\r
+  IN CHAR16                       *VarName,\r
+  OUT UINT64                      *VarValue\r
+  )\r
+{\r
+  UINT8                        *TmpBuffer;\r
+  CHAR16                       *StringPtr;\r
+  UINTN                        Length;\r
+  EFI_STATUS                   Status;\r
+\r
+  //\r
+  // Find VarName related string.\r
+  //\r
+  StringPtr = StrStr (ConfigElement, VarName);\r
+  ASSERT (StringPtr != NULL);\r
+\r
+  //\r
+  // Skip the "VarName=" string\r
+  //\r
+  StringPtr += StrLen (VarName) + 1;\r
+\r
+  //\r
+  // Get Offset\r
+  //\r
+  Status = InternalHiiGetValueOfNumber (StringPtr, &TmpBuffer, &Length);\r
+  if (EFI_ERROR (Status)) {\r
+    return Status;\r
+  }\r
+\r
+  *VarValue = 0;\r
+  CopyMem (VarValue, TmpBuffer, (((Length + 1) / 2) < sizeof (UINT64)) ? ((Length + 1) / 2) : sizeof (UINT64));\r
+\r
+  FreePool (TmpBuffer);\r
+\r
+  return EFI_SUCCESS;\r
+}\r
+\r
 /**\r
   This internal function parses IFR data to validate current setting.\r
 \r
-  @param ConfigResp         ConfigResp string contains the current setting.\r
+  Base on the NameValueType, if it is TRUE, RequestElement and HiiHandle is valid;\r
+  else the VarBuffer and CurrentBlockArray is valid.\r
+\r
   @param HiiPackageList     Point to Hii package list.\r
   @param PackageListLength  The length of the pacakge.\r
   @param VarGuid            Guid of the buffer storage.\r
   @param VarName            Name of the buffer storage.\r
+  @param VarBuffer          The data buffer for the storage.\r
+  @param CurrentBlockArray  The block array from the config Requst string.\r
+  @param RequestElement     The config string for this storage.\r
+  @param HiiHandle          The HiiHandle for this formset.\r
+  @param NameValueType      Whether current storage is name/value varstore or not.\r
   \r
   @retval EFI_SUCCESS            The current setting is valid.\r
   @retval EFI_OUT_OF_RESOURCES   The memory is not enough.\r
   @retval EFI_INVALID_PARAMETER  The config string or the Hii package is invalid.\r
 **/\r
 EFI_STATUS\r
-EFIAPI\r
-InternalHiiValidateCurrentSetting (\r
-  IN EFI_STRING                    ConfigResp,\r
+ValidateQuestionFromVfr (\r
   IN EFI_HII_PACKAGE_LIST_HEADER   *HiiPackageList,\r
   IN UINTN                         PackageListLength,\r
   IN EFI_GUID                      *VarGuid,\r
-  IN CHAR16                        *VarName\r
+  IN CHAR16                        *VarName,\r
+  IN UINT8                         *VarBuffer,\r
+  IN IFR_BLOCK_DATA                *CurrentBlockArray,\r
+  IN CHAR16                        *RequestElement,\r
+  IN EFI_HII_HANDLE                HiiHandle,\r
+  IN BOOLEAN                       NameValueType\r
   )\r
-{ \r
-  IFR_BLOCK_DATA               *CurrentBlockArray;\r
-  IFR_BLOCK_DATA               *BlockData;\r
-  IFR_BLOCK_DATA               *NewBlockData;\r
+{\r
   IFR_BLOCK_DATA               VarBlockData;\r
-  EFI_STRING                   StringPtr;\r
-  UINTN                        Length;\r
-  UINT8                        *TmpBuffer;\r
   UINT16                       Offset;\r
   UINT16                       Width;\r
   UINT64                       VarValue;\r
-  LIST_ENTRY                   *Link;\r
-  UINT8                        *VarBuffer;\r
-  UINTN                        MaxBufferSize;\r
+  EFI_IFR_TYPE_VALUE           TmpValue;\r
   EFI_STATUS                   Status;\r
-  EFI_HII_PACKAGE_HEADER       PacakgeHeader;\r
+  EFI_HII_PACKAGE_HEADER       PackageHeader;\r
   UINT32                       PackageOffset;\r
   UINT8                        *PackageData;\r
   UINTN                        IfrOffset;\r
   EFI_IFR_OP_HEADER            *IfrOpHdr;\r
   EFI_IFR_VARSTORE             *IfrVarStore;\r
+  EFI_IFR_VARSTORE_NAME_VALUE  *IfrNameValueStore;\r
+  EFI_IFR_VARSTORE_EFI         *IfrEfiVarStore;\r
+  IFR_VARSTORAGE_DATA          VarStoreData;\r
   EFI_IFR_ONE_OF               *IfrOneOf;\r
   EFI_IFR_NUMERIC              *IfrNumeric;\r
   EFI_IFR_ONE_OF_OPTION        *IfrOneOfOption;\r
@@ -963,268 +1167,135 @@ InternalHiiValidateCurrentSetting (
   EFI_IFR_STRING               *IfrString;\r
   CHAR8                        *VarStoreName;\r
   UINTN                        Index;\r
-  \r
-  //\r
-  // 1. Get the current setting to current block data array and Convert them into VarBuffer\r
-  //\r
+  CHAR16                       *QuestionName;\r
+  CHAR16                       *StringPtr;\r
 \r
-  //\r
-  // Skip ConfigHdr string\r
-  //\r
-  StringPtr = ConfigResp;\r
-  StringPtr = StrStr (ConfigResp, L"&OFFSET");\r
-  if (StringPtr == NULL) {\r
-    //\r
-    // No ConfigBlock value is required to be validated.\r
-    // EFI_SUCCESS directly return.\r
-    //\r
-    return EFI_SUCCESS;\r
-  }\r
-   \r
   //\r
   // Initialize the local variables.\r
   //\r
-  Index         = 0;\r
-  VarStoreName  = NULL;\r
-  Status        = EFI_SUCCESS;\r
-  BlockData     = NULL;\r
-  NewBlockData  = NULL;\r
-  TmpBuffer     = NULL;\r
-  MaxBufferSize = HII_LIB_DEFAULT_VARSTORE_SIZE;\r
-  VarBuffer     = AllocateZeroPool (MaxBufferSize);\r
-  if (VarBuffer == NULL) {\r
-    return EFI_OUT_OF_RESOURCES;\r
-  }\r
+  Index             = 0;\r
+  VarStoreName      = NULL;\r
+  Status            = EFI_SUCCESS;\r
+  VarValue          = 0;\r
+  IfrVarStore       = NULL;\r
+  IfrNameValueStore = NULL;\r
+  IfrEfiVarStore    = NULL;\r
+  ZeroMem (&VarStoreData, sizeof (IFR_VARSTORAGE_DATA));\r
+  ZeroMem (&VarBlockData, sizeof (VarBlockData));\r
 \r
   //\r
-  // Init CurrentBlockArray\r
-  //\r
-  CurrentBlockArray = (IFR_BLOCK_DATA *) AllocateZeroPool (sizeof (IFR_BLOCK_DATA));\r
-  if (CurrentBlockArray == NULL) {\r
-    Status = EFI_OUT_OF_RESOURCES;\r
-    goto Done;\r
-  }\r
-  InitializeListHead (&CurrentBlockArray->Entry);\r
-  \r
-  //\r
-  // Parse each <RequestElement> if exists\r
-  // Only <BlockName> format is supported by this help function.\r
-  // <BlockName> ::= &'OFFSET='<Number>&'WIDTH='<Number>\r
+  // Check IFR value is in block data, then Validate Value\r
   //\r
-  while (*StringPtr != 0 && StrnCmp (StringPtr, L"&OFFSET=", StrLen (L"&OFFSET=")) == 0) {\r
-    //\r
-    // Skip the &OFFSET= string\r
-    // \r
-    StringPtr += StrLen (L"&OFFSET=");\r
+  PackageOffset = sizeof (EFI_HII_PACKAGE_LIST_HEADER);\r
+  while (PackageOffset < PackageListLength) {\r
+    CopyMem (&PackageHeader, (UINT8 *) HiiPackageList + PackageOffset, sizeof (PackageHeader));\r
 \r
     //\r
-    // Get Offset\r
+    // Parse IFR opcode from the form package.\r
     //\r
-    Status = InternalHiiGetValueOfNumber (StringPtr, &TmpBuffer, &Length);\r
-    if (EFI_ERROR (Status)) {\r
-      goto Done;\r
-    }\r
-    Offset = 0;\r
-    CopyMem (\r
-      &Offset,\r
-      TmpBuffer,\r
-      (((Length + 1) / 2) < sizeof (UINT16)) ? ((Length + 1) / 2) : sizeof (UINT16)\r
-      );\r
-    FreePool (TmpBuffer);\r
-    TmpBuffer = NULL;\r
+    if (PackageHeader.Type == EFI_HII_PACKAGE_FORMS) {\r
+      IfrOffset   = sizeof (PackageHeader);\r
+      PackageData = (UINT8 *) HiiPackageList + PackageOffset;\r
+      while (IfrOffset < PackageHeader.Length) {\r
+        IfrOpHdr = (EFI_IFR_OP_HEADER *) (PackageData + IfrOffset);\r
+        //\r
+        // Validate current setting to the value built in IFR opcode\r
+        //\r
+        switch (IfrOpHdr->OpCode) {\r
+        case EFI_IFR_VARSTORE_OP:\r
+          //\r
+          // VarStoreId has been found. No further found.\r
+          //\r
+          if (VarStoreData.VarStoreId != 0) {\r
+            break;\r
+          }\r
+          //\r
+          // Find the matched VarStoreId to the input VarGuid and VarName\r
+          //\r
+          IfrVarStore = (EFI_IFR_VARSTORE *) IfrOpHdr;\r
+          if (CompareGuid ((EFI_GUID *) (VOID *) &IfrVarStore->Guid, VarGuid)) {\r
+            VarStoreName = (CHAR8 *) IfrVarStore->Name;\r
+            for (Index = 0; VarStoreName[Index] != 0; Index ++) {\r
+              if ((CHAR16) VarStoreName[Index] != VarName[Index]) {\r
+                break;\r
+              }\r
+            }\r
+            //\r
+            // The matched VarStore is found.\r
+            //\r
+            if ((VarStoreName[Index] != 0) || (VarName[Index] != 0)) {\r
+              IfrVarStore = NULL;\r
+            }\r
+          } else {\r
+            IfrVarStore = NULL;\r
+          }\r
 \r
-    StringPtr += Length;\r
-    if (StrnCmp (StringPtr, L"&WIDTH=", StrLen (L"&WIDTH=")) != 0) {\r
-      Status = EFI_INVALID_PARAMETER;\r
-      goto Done;\r
-    }\r
-    StringPtr += StrLen (L"&WIDTH=");\r
+          if (IfrVarStore != NULL) {\r
+            VarStoreData.VarStoreId = IfrVarStore->VarStoreId;\r
+            VarStoreData.Size       = IfrVarStore->Size;\r
+          }\r
+          break;\r
+        case EFI_IFR_VARSTORE_NAME_VALUE_OP:\r
+          //\r
+          // VarStoreId has been found. No further found.\r
+          //\r
+          if (VarStoreData.VarStoreId != 0) {\r
+            break;\r
+          }\r
+          //\r
+          // Find the matched VarStoreId to the input VarGuid\r
+          //\r
+          IfrNameValueStore = (EFI_IFR_VARSTORE_NAME_VALUE *) IfrOpHdr;\r
+          if (!CompareGuid ((EFI_GUID *) (VOID *) &IfrNameValueStore->Guid, VarGuid)) {\r
+            IfrNameValueStore = NULL;\r
+          }\r
 \r
-    //\r
-    // Get Width\r
-    //\r
-    Status = InternalHiiGetValueOfNumber (StringPtr, &TmpBuffer, &Length);\r
-    if (EFI_ERROR (Status)) {\r
-      goto Done;\r
-    }\r
-    Width = 0;\r
-    CopyMem (\r
-      &Width,\r
-      TmpBuffer,\r
-      (((Length + 1) / 2) < sizeof (UINT16)) ? ((Length + 1) / 2) : sizeof (UINT16)\r
-      );\r
-    FreePool (TmpBuffer);\r
-    TmpBuffer = NULL;\r
+          if (IfrNameValueStore != NULL) {\r
+            VarStoreData.VarStoreId = IfrNameValueStore->VarStoreId;\r
+          }\r
+          break;\r
+        case EFI_IFR_VARSTORE_EFI_OP:\r
+          //\r
+          // VarStore is found. Don't need to search any more.\r
+          //\r
+          if (VarStoreData.VarStoreId != 0) {\r
+            break;\r
+          }\r
 \r
-    StringPtr += Length;\r
-    if (*StringPtr != 0 && *StringPtr != L'&') {\r
-      Status = EFI_INVALID_PARAMETER;\r
-      goto Done;\r
-    }\r
+          IfrEfiVarStore = (EFI_IFR_VARSTORE_EFI *) IfrOpHdr;\r
 \r
-    if (StrnCmp (StringPtr, L"&VALUE=", StrLen (L"&VALUE=")) != 0) {\r
-      Status = EFI_INVALID_PARAMETER;\r
-      goto Done;\r
-    }\r
-    StringPtr += StrLen (L"&VALUE=");\r
+          //\r
+          // If the length is small than the structure, this is from old efi \r
+          // varstore definition. Old efi varstore get config directly from \r
+          // GetVariable function.\r
+          //\r
+          if (IfrOpHdr->Length < sizeof (EFI_IFR_VARSTORE_EFI)) {\r
+            break;\r
+          }\r
 \r
-    //\r
-    // Get Value\r
-    //\r
-    Status = InternalHiiGetValueOfNumber (StringPtr, &TmpBuffer, &Length);\r
-    if (EFI_ERROR (Status)) {\r
-      goto Done;\r
-    }\r
+          if (CompareGuid ((EFI_GUID *) (VOID *) &IfrEfiVarStore->Guid, VarGuid)) {\r
+            VarStoreName = (CHAR8 *) IfrEfiVarStore->Name;\r
+            for (Index = 0; VarStoreName[Index] != 0; Index ++) {\r
+              if ((CHAR16) VarStoreName[Index] != VarName[Index]) {\r
+                break;\r
+              }\r
+            }\r
+            //\r
+            // The matched VarStore is found.\r
+            //\r
+            if ((VarStoreName[Index] != 0) || (VarName[Index] != 0)) {\r
+              IfrEfiVarStore = NULL;\r
+            }\r
+          } else {\r
+            IfrEfiVarStore = NULL;\r
+          }\r
 \r
-    StringPtr += Length;\r
-    if (*StringPtr != 0 && *StringPtr != L'&') {\r
-      Status = EFI_INVALID_PARAMETER;\r
-      goto Done;\r
-    }\r
-\r
-    //\r
-    // Check whether VarBuffer is enough\r
-    //\r
-    if ((UINTN) (Offset + Width) > MaxBufferSize) {\r
-      VarBuffer = ReallocatePool (\r
-                    MaxBufferSize,\r
-                    Offset + Width + HII_LIB_DEFAULT_VARSTORE_SIZE,\r
-                    VarBuffer\r
-                    );\r
-      if (VarBuffer == NULL) {\r
-        Status = EFI_OUT_OF_RESOURCES;\r
-        goto Done;\r
-      }\r
-      MaxBufferSize = Offset + Width + HII_LIB_DEFAULT_VARSTORE_SIZE;\r
-    }\r
-\r
-    //\r
-    // Update the Block with configuration info\r
-    //\r
-    CopyMem (VarBuffer + Offset, TmpBuffer, Width);\r
-    FreePool (TmpBuffer);\r
-    TmpBuffer = NULL;\r
-\r
-    //\r
-    // Set new Block Data\r
-    //\r
-    NewBlockData = (IFR_BLOCK_DATA *) AllocateZeroPool (sizeof (IFR_BLOCK_DATA));\r
-    if (NewBlockData == NULL) {\r
-      Status = EFI_OUT_OF_RESOURCES;\r
-      goto Done;\r
-    }\r
-    NewBlockData->Offset = Offset;\r
-    NewBlockData->Width  = Width;\r
-\r
-    //\r
-    // Insert the new block data into the block data array.\r
-    //\r
-    for (Link = CurrentBlockArray->Entry.ForwardLink; Link != &CurrentBlockArray->Entry; Link = Link->ForwardLink) {\r
-      BlockData = BASE_CR (Link, IFR_BLOCK_DATA, Entry);\r
-      if (NewBlockData->Offset == BlockData->Offset) {\r
-        if (NewBlockData->Width > BlockData->Width) {\r
-          BlockData->Width = NewBlockData->Width;\r
-        }\r
-        FreePool (NewBlockData);\r
-        break;\r
-      } else if (NewBlockData->Offset < BlockData->Offset) {\r
-        //\r
-        // Insert new block data as the previous one of this link.\r
-        //\r
-        InsertTailList (Link, &NewBlockData->Entry);\r
-        break;\r
-      }\r
-    }\r
-\r
-    //\r
-    // Insert new block data into the array tail.\r
-    //\r
-    if (Link == &CurrentBlockArray->Entry) {\r
-      InsertTailList (Link, &NewBlockData->Entry);\r
-    }\r
-\r
-    //\r
-    // If '\0', parsing is finished. \r
-    //\r
-    if (*StringPtr == 0) {\r
-      break;\r
-    }\r
-    //\r
-    // Go to next ConfigBlock \r
-    //\r
-  }\r
-\r
-  //\r
-  // Merge the aligned block data into the single block data.\r
-  //\r
-  Link = CurrentBlockArray->Entry.ForwardLink;\r
-  while ((Link != &CurrentBlockArray->Entry) && (Link->ForwardLink != &CurrentBlockArray->Entry)) {\r
-    BlockData = BASE_CR (Link, IFR_BLOCK_DATA, Entry);\r
-    NewBlockData = BASE_CR (Link->ForwardLink, IFR_BLOCK_DATA, Entry);\r
-    if ((NewBlockData->Offset >= BlockData->Offset) && (NewBlockData->Offset <= (BlockData->Offset + BlockData->Width))) {\r
-      if ((NewBlockData->Offset + NewBlockData->Width) > (BlockData->Offset + BlockData->Width)) {\r
-        BlockData->Width = (UINT16) (NewBlockData->Offset + NewBlockData->Width - BlockData->Offset);\r
-      }\r
-      RemoveEntryList (Link->ForwardLink);\r
-      FreePool (NewBlockData);\r
-      continue;\r
-    }\r
-    Link = Link->ForwardLink;      \r
-  }\r
-  \r
-  if (IsListEmpty (&CurrentBlockArray->Entry)) {\r
-    Status = EFI_SUCCESS;\r
-    goto Done;\r
-  }\r
-\r
-  //\r
-  // 2. Check IFR value is in block data, then Validate Value\r
-  //\r
-  ZeroMem (&VarBlockData, sizeof (VarBlockData));\r
-  VarValue      = 0;\r
-  IfrVarStore   = NULL;\r
-  PackageOffset = sizeof (EFI_HII_PACKAGE_LIST_HEADER);\r
-  while (PackageOffset < PackageListLength) {\r
-    CopyMem (&PacakgeHeader, (UINT8 *) HiiPackageList + PackageOffset, sizeof (PacakgeHeader));\r
-    \r
-    //\r
-    // Parse IFR opcode from the form package.\r
-    //\r
-    if (PacakgeHeader.Type == EFI_HII_PACKAGE_FORMS) {\r
-      IfrOffset   = sizeof (PacakgeHeader);\r
-      PackageData = (UINT8 *) HiiPackageList + PackageOffset;\r
-      while (IfrOffset < PacakgeHeader.Length) {\r
-        IfrOpHdr = (EFI_IFR_OP_HEADER *) (PackageData + IfrOffset);\r
-        //\r
-        // Validate current setting to the value built in IFR opcode\r
-        //\r
-        switch (IfrOpHdr->OpCode) {\r
-        case EFI_IFR_VARSTORE_OP:          \r
-          //\r
-          // VarStoreId has been found. No further found.\r
-          //\r
-          if (IfrVarStore != NULL) {\r
-            break;\r
-          }\r
-          //\r
-          // Find the matched VarStoreId to the input VarGuid and VarName\r
-          //\r
-          IfrVarStore = (EFI_IFR_VARSTORE *) IfrOpHdr;\r
-          if (CompareGuid ((EFI_GUID *) (VOID *) &IfrVarStore->Guid, VarGuid)) {\r
-            VarStoreName = (CHAR8 *) IfrVarStore->Name;\r
-            for (Index = 0; VarStoreName[Index] != 0; Index ++) {\r
-              if ((CHAR16) VarStoreName[Index] != VarName[Index]) {\r
-                break;\r
-              }\r
-            }\r
+          if (IfrEfiVarStore != NULL) {\r
             //\r
-            // The matched VarStore is found.\r
+            // Find the matched VarStore\r
             //\r
-            if ((VarStoreName[Index] != 0) || (VarName[Index] != 0)) {\r
-              IfrVarStore = NULL;\r
-            }\r
-          } else {\r
-            IfrVarStore = NULL;\r
+            VarStoreData.VarStoreId = IfrEfiVarStore->VarStoreId;\r
+            VarStoreData.Size       = IfrEfiVarStore->Size;\r
           }\r
           break;\r
         case EFI_IFR_FORM_OP:\r
@@ -1232,9 +1303,8 @@ InternalHiiValidateCurrentSetting (
           //\r
           // Check the matched VarStoreId is found.\r
           //\r
-          if (IfrVarStore == NULL) {\r
-            Status = EFI_SUCCESS;\r
-            goto Done;\r
+          if (VarStoreData.VarStoreId == 0) {\r
+            return EFI_SUCCESS;\r
           }\r
           break;\r
         case EFI_IFR_ONE_OF_OP:\r
@@ -1245,53 +1315,66 @@ InternalHiiValidateCurrentSetting (
           //\r
           // OneOf question is not in IFR Form. This IFR form is not valid. \r
           //\r
-          if (IfrVarStore == NULL) {\r
-            Status = EFI_INVALID_PARAMETER;\r
-            goto Done;\r
+          if (VarStoreData.VarStoreId == 0) {\r
+            return EFI_INVALID_PARAMETER;\r
           }\r
           // \r
           // Check whether this question is for the requested varstore.\r
           //\r
           IfrOneOf = (EFI_IFR_ONE_OF *) IfrOpHdr;\r
-          if (IfrOneOf->Question.VarStoreId != IfrVarStore->VarStoreId) {\r
+          if (IfrOneOf->Question.VarStoreId != VarStoreData.VarStoreId) {\r
             break;\r
           }\r
-          \r
-          //\r
-          // Get Offset by Question header and Width by DataType Flags\r
-          //\r
-          Offset = IfrOneOf->Question.VarStoreInfo.VarOffset;\r
-          Width  = (UINT16) (1 << (IfrOneOf->Flags & EFI_IFR_NUMERIC_SIZE));\r
-          //\r
-          // Check whether this question is in current block array.\r
-          //\r
-          if (!BlockArrayCheck (CurrentBlockArray, Offset, Width)) {\r
+\r
+          if (NameValueType) {\r
+            QuestionName = HiiGetString (HiiHandle, IfrOneOf->Question.VarStoreInfo.VarName, NULL);\r
+            ASSERT (QuestionName != NULL);\r
+\r
+            if (StrStr (RequestElement, QuestionName) == NULL) {\r
+              //\r
+              // This question is not in the current configuration string. Skip it.\r
+              //\r
+              break;\r
+            }\r
+\r
+            Status = GetValueFromRequest (RequestElement, QuestionName, &VarValue);\r
+            if (EFI_ERROR (Status)) {\r
+              return Status;\r
+            }\r
+          } else {\r
             //\r
-            // This question is not in the current configuration string. Skip it.\r
+            // Get Offset by Question header and Width by DataType Flags\r
             //\r
-            break;\r
-          }\r
-          //\r
-          // Check this var question is in the var storage \r
-          //\r
-          if ((Offset + Width) > IfrVarStore->Size) {\r
+            Offset = IfrOneOf->Question.VarStoreInfo.VarOffset;\r
+            Width  = (UINT16) (1 << (IfrOneOf->Flags & EFI_IFR_NUMERIC_SIZE));\r
             //\r
-            // This question exceeds the var store size. \r
+            // Check whether this question is in current block array.\r
             //\r
-            Status = EFI_INVALID_PARAMETER;\r
-            goto Done;\r
-          }\r
+            if (!BlockArrayCheck (CurrentBlockArray, Offset, Width)) {\r
+              //\r
+              // This question is not in the current configuration string. Skip it.\r
+              //\r
+              break;\r
+            }\r
+            //\r
+            // Check this var question is in the var storage \r
+            //\r
+            if ((Offset + Width) > VarStoreData.Size) {\r
+              //\r
+              // This question exceeds the var store size. \r
+              //\r
+              return EFI_INVALID_PARAMETER;\r
+            }\r
 \r
-          //\r
-          // Get the current value for oneof opcode\r
-          //\r
-          VarValue = 0;\r
-          CopyMem (&VarValue, VarBuffer +  Offset, Width);\r
+            //\r
+            // Get the current value for oneof opcode\r
+            //\r
+            VarValue = 0;\r
+            CopyMem (&VarValue, VarBuffer +  Offset, Width);\r
+          }\r
           //\r
           // Set Block Data, to be checked in the following Oneof option opcode.\r
           //\r
-          VarBlockData.Offset     = Offset;\r
-          VarBlockData.Width      = Width;\r
           VarBlockData.OpCode     = IfrOpHdr->OpCode;\r
           VarBlockData.Scope      = IfrOpHdr->Scope;\r
           break;\r
@@ -1303,266 +1386,680 @@ InternalHiiValidateCurrentSetting (
           //\r
           // Numeric question is not in IFR Form. This IFR form is not valid. \r
           //\r
-          if (IfrVarStore == NULL) {\r
-            Status = EFI_INVALID_PARAMETER;\r
-            goto Done;\r
+          if (VarStoreData.VarStoreId == 0) {\r
+            return EFI_INVALID_PARAMETER;\r
           }\r
           //\r
           // Check whether this question is for the requested varstore.\r
           //\r
           IfrNumeric = (EFI_IFR_NUMERIC *) IfrOpHdr;\r
-          if (IfrNumeric->Question.VarStoreId != IfrVarStore->VarStoreId) {\r
+          if (IfrNumeric->Question.VarStoreId != VarStoreData.VarStoreId) {\r
             break;\r
           }\r
-          \r
-          //\r
-          // Get Offset by Question header and Width by DataType Flags\r
-          //\r
-          Offset = IfrNumeric->Question.VarStoreInfo.VarOffset;\r
-          Width  = (UINT16) (1 << (IfrNumeric->Flags & EFI_IFR_NUMERIC_SIZE));\r
-          //\r
-          // Check whether this question is in current block array.\r
-          //\r
-          if (!BlockArrayCheck (CurrentBlockArray, Offset, Width)) {\r
+\r
+          if (NameValueType) {\r
+            QuestionName = HiiGetString (HiiHandle, IfrNumeric->Question.VarStoreInfo.VarName, NULL);\r
+            ASSERT (QuestionName != NULL);\r
+\r
+            if (StrStr (RequestElement, QuestionName) == NULL) {\r
+              //\r
+              // This question is not in the current configuration string. Skip it.\r
+              //\r
+              break;\r
+            }\r
+            \r
+            Status = GetValueFromRequest (RequestElement, QuestionName, &VarValue);\r
+            if (EFI_ERROR (Status)) {\r
+              return Status;\r
+            }\r
+          } else {\r
             //\r
-            // This question is not in the current configuration string. Skip it.\r
+            // Get Offset by Question header and Width by DataType Flags\r
             //\r
-            break;\r
+            Offset = IfrNumeric->Question.VarStoreInfo.VarOffset;\r
+            Width  = (UINT16) (1 << (IfrNumeric->Flags & EFI_IFR_NUMERIC_SIZE));\r
+            //\r
+            // Check whether this question is in current block array.\r
+            //\r
+            if (!BlockArrayCheck (CurrentBlockArray, Offset, Width)) {\r
+              //\r
+              // This question is not in the current configuration string. Skip it.\r
+              //\r
+              break;\r
+            }\r
+            //\r
+            // Check this var question is in the var storage \r
+            //\r
+            if ((Offset + Width) > VarStoreData.Size) {\r
+              //\r
+              // This question exceeds the var store size. \r
+              //\r
+              return EFI_INVALID_PARAMETER;\r
+            }\r
+\r
+            //\r
+            // Check the current value is in the numeric range.\r
+            //\r
+            VarValue = 0;\r
+            CopyMem (&VarValue, VarBuffer +  Offset, Width);\r
           }\r
+          if ((IfrNumeric->Flags & EFI_IFR_DISPLAY) == 0) {\r
+            switch (IfrNumeric->Flags & EFI_IFR_NUMERIC_SIZE) {\r
+            case EFI_IFR_NUMERIC_SIZE_1:\r
+              if ((INT8) VarValue < (INT8) IfrNumeric->data.u8.MinValue || (INT8) VarValue > (INT8) IfrNumeric->data.u8.MaxValue) {\r
+                //\r
+                // Not in the valid range.\r
+                //\r
+                return EFI_INVALID_PARAMETER;\r
+              }\r
+              break;\r
+            case EFI_IFR_NUMERIC_SIZE_2:\r
+              if ((INT16) VarValue < (INT16) IfrNumeric->data.u16.MinValue || (INT16) VarValue > (INT16) IfrNumeric->data.u16.MaxValue) {\r
+                //\r
+                // Not in the valid range.\r
+                //\r
+                return EFI_INVALID_PARAMETER;\r
+              }\r
+              break;\r
+            case EFI_IFR_NUMERIC_SIZE_4:\r
+              if ((INT32) VarValue < (INT32) IfrNumeric->data.u32.MinValue || (INT32) VarValue > (INT32) IfrNumeric->data.u32.MaxValue) {\r
+                //\r
+                // Not in the valid range.\r
+                //\r
+                return EFI_INVALID_PARAMETER;\r
+              }\r
+              break;\r
+            case EFI_IFR_NUMERIC_SIZE_8:\r
+              if ((INT64) VarValue < (INT64) IfrNumeric->data.u64.MinValue || (INT64) VarValue > (INT64) IfrNumeric->data.u64.MaxValue) {\r
+                //\r
+                // Not in the valid range.\r
+                //\r
+                return EFI_INVALID_PARAMETER;\r
+              }\r
+              break;\r
+            }\r
+          } else {\r
+            switch (IfrNumeric->Flags & EFI_IFR_NUMERIC_SIZE) {\r
+            case EFI_IFR_NUMERIC_SIZE_1:\r
+              if ((UINT8) VarValue < IfrNumeric->data.u8.MinValue || (UINT8) VarValue > IfrNumeric->data.u8.MaxValue) {\r
+                //\r
+                // Not in the valid range.\r
+                //\r
+                return EFI_INVALID_PARAMETER;\r
+              }\r
+              break;\r
+            case EFI_IFR_NUMERIC_SIZE_2:\r
+              if ((UINT16) VarValue < IfrNumeric->data.u16.MinValue || (UINT16) VarValue > IfrNumeric->data.u16.MaxValue) {\r
+                //\r
+                // Not in the valid range.\r
+                //\r
+                return EFI_INVALID_PARAMETER;\r
+              }\r
+              break;\r
+            case EFI_IFR_NUMERIC_SIZE_4:\r
+              if ((UINT32) VarValue < IfrNumeric->data.u32.MinValue || (UINT32) VarValue > IfrNumeric->data.u32.MaxValue) {\r
+                //\r
+                // Not in the valid range.\r
+                //\r
+                return EFI_INVALID_PARAMETER;\r
+              }\r
+              break;\r
+            case EFI_IFR_NUMERIC_SIZE_8:\r
+              if ((UINT64) VarValue < IfrNumeric->data.u64.MinValue || (UINT64) VarValue > IfrNumeric->data.u64.MaxValue) {\r
+                //\r
+                // Not in the valid range.\r
+                //\r
+                return EFI_INVALID_PARAMETER;\r
+              }\r
+              break;\r
+            }\r
+          }\r
+          break;\r
+        case EFI_IFR_CHECKBOX_OP:\r
           //\r
-          // Check this var question is in the var storage \r
+          // Check value is BOOLEAN type, only 0 and 1 is valid.\r
           //\r
-          if ((Offset + Width) > IfrVarStore->Size) {\r
-            //\r
-            // This question exceeds the var store size\r
-            //\r
-            Status = EFI_INVALID_PARAMETER;\r
-            goto Done;\r
+\r
+          //\r
+          // CheckBox question is not in IFR Form. This IFR form is not valid\r
+          //\r
+          if (VarStoreData.VarStoreId == 0) {\r
+            return EFI_INVALID_PARAMETER;\r
           }\r
 \r
           //\r
-          // Check the current value is in the numeric range.\r
+          // Check whether this question is for the requested varstore.\r
           //\r
-          VarValue = 0;\r
-          CopyMem (&VarValue, VarBuffer +  Offset, Width);\r
-          switch (IfrNumeric->Flags & EFI_IFR_NUMERIC_SIZE) {\r
-          case EFI_IFR_NUMERIC_SIZE_1:\r
-            if ((UINT8) VarValue < IfrNumeric->data.u8.MinValue || (UINT8) VarValue > IfrNumeric->data.u8.MaxValue) {\r
-              //\r
-              // Not in the valid range.\r
-              //\r
-              Status = EFI_INVALID_PARAMETER;\r
-              goto Done;\r
-            }\r
+          IfrCheckBox = (EFI_IFR_CHECKBOX *) IfrOpHdr;\r
+          if (IfrCheckBox->Question.VarStoreId != VarStoreData.VarStoreId) {\r
             break;\r
-          case EFI_IFR_NUMERIC_SIZE_2:\r
-            if ((UINT16) VarValue < IfrNumeric->data.u16.MinValue || (UINT16) VarValue > IfrNumeric->data.u16.MaxValue) {\r
+          }\r
+\r
+          if (NameValueType) {\r
+            QuestionName = HiiGetString (HiiHandle, IfrCheckBox->Question.VarStoreInfo.VarName, NULL);\r
+            ASSERT (QuestionName != NULL);\r
+\r
+            if (StrStr (RequestElement, QuestionName) == NULL) {\r
               //\r
-              // Not in the valid range.\r
+              // This question is not in the current configuration string. Skip it.\r
               //\r
-              Status = EFI_INVALID_PARAMETER;\r
-              goto Done;\r
+              break;\r
             }\r
-            break;\r
-          case EFI_IFR_NUMERIC_SIZE_4:\r
-            if ((UINT32) VarValue < IfrNumeric->data.u32.MinValue || (UINT32) VarValue > IfrNumeric->data.u32.MaxValue) {\r
+            \r
+            Status = GetValueFromRequest (RequestElement, QuestionName, &VarValue);\r
+            if (EFI_ERROR (Status)) {\r
+              return Status;\r
+            }\r
+          } else {\r
+            //\r
+            // Get Offset by Question header\r
+            //\r
+            Offset = IfrCheckBox->Question.VarStoreInfo.VarOffset;\r
+            Width  = (UINT16) sizeof (BOOLEAN);\r
+            //\r
+            // Check whether this question is in current block array.\r
+            //\r
+            if (!BlockArrayCheck (CurrentBlockArray, Offset, Width)) {\r
               //\r
-              // Not in the valid range.\r
+              // This question is not in the current configuration string. Skip it.\r
               //\r
-              Status = EFI_INVALID_PARAMETER;\r
-              goto Done;\r
+              break;\r
             }\r
-            break;\r
-          case EFI_IFR_NUMERIC_SIZE_8:\r
-            if ((UINT64) VarValue < IfrNumeric->data.u64.MinValue || (UINT64) VarValue > IfrNumeric->data.u64.MaxValue) {\r
+            //\r
+            // Check this var question is in the var storage \r
+            //\r
+            if ((Offset + Width) > VarStoreData.Size) {\r
               //\r
-              // Not in the valid range.\r
+              // This question exceeds the var store size. \r
               //\r
-              Status = EFI_INVALID_PARAMETER;\r
-              goto Done;\r
+              return EFI_INVALID_PARAMETER;\r
             }\r
-            break;\r
+            //\r
+            // Check the current value is in the numeric range.\r
+            //\r
+            VarValue = 0;\r
+            CopyMem (&VarValue, VarBuffer +  Offset, Width);\r
+          }\r
+          //\r
+          // Boolean type, only 1 and 0 is valid.\r
+          //\r
+          if (VarValue > 1) {\r
+            return EFI_INVALID_PARAMETER;\r
           }\r
-\r
           break;\r
-        case EFI_IFR_CHECKBOX_OP:\r
+        case EFI_IFR_STRING_OP:\r
           //\r
-          // Check value is BOOLEAN type, only 0 and 1 is valid.\r
+          // Check current string length is less than maxsize\r
           //\r
 \r
           //\r
           // CheckBox question is not in IFR Form. This IFR form is not valid. \r
           //\r
-          if (IfrVarStore == NULL) {\r
-            Status = EFI_INVALID_PARAMETER;\r
-            goto Done;\r
+          if (VarStoreData.VarStoreId == 0) {\r
+            return EFI_INVALID_PARAMETER;\r
           }\r
 \r
           //\r
           // Check whether this question is for the requested varstore.\r
           //\r
-          IfrCheckBox = (EFI_IFR_CHECKBOX *) IfrOpHdr;\r
-          if (IfrCheckBox->Question.VarStoreId != IfrVarStore->VarStoreId) {\r
+          IfrString = (EFI_IFR_STRING *) IfrOpHdr;\r
+          if (IfrString->Question.VarStoreId != VarStoreData.VarStoreId) {\r
             break;\r
           }\r
-          \r
-          //\r
-          // Get Offset by Question header\r
           //\r
-          Offset = IfrCheckBox->Question.VarStoreInfo.VarOffset;\r
-          Width  = (UINT16) sizeof (BOOLEAN);\r
+          // Get Width by OneOf Flags\r
           //\r
-          // Check whether this question is in current block array.\r
-          //\r
-          if (!BlockArrayCheck (CurrentBlockArray, Offset, Width)) {\r
+          Width  = (UINT16) (IfrString->MaxSize * sizeof (UINT16));\r
+          if (NameValueType) {\r
+            QuestionName = HiiGetString (HiiHandle, IfrString->Question.VarStoreInfo.VarName, NULL);\r
+            ASSERT (QuestionName != NULL);\r
+\r
+            StringPtr = StrStr (RequestElement, QuestionName);\r
+            if (StringPtr == NULL) {\r
+              //\r
+              // This question is not in the current configuration string. Skip it.\r
+              //\r
+              break;\r
+            }\r
+\r
             //\r
-            // This question is not in the current configuration string. Skip it.\r
+            // Skip the "=".\r
+            // \r
+            StringPtr += 1;\r
+            \r
             //\r
-            break;\r
+            // Check current string length is less than maxsize\r
+            //\r
+            if (StrSize (StringPtr) > Width) {\r
+              return EFI_INVALID_PARAMETER;\r
+            }\r
+          } else {\r
+            //\r
+            // Get Offset/Width by Question header and OneOf Flags\r
+            //\r
+            Offset = IfrString->Question.VarStoreInfo.VarOffset;\r
+            //\r
+            // Check whether this question is in current block array.\r
+            //\r
+            if (!BlockArrayCheck (CurrentBlockArray, Offset, Width)) {\r
+              //\r
+              // This question is not in the current configuration string. Skip it.\r
+              //\r
+              break;\r
+            }\r
+            //\r
+            // Check this var question is in the var storage \r
+            //\r
+            if ((Offset + Width) > VarStoreData.Size) {\r
+              //\r
+              // This question exceeds the var store size. \r
+              //\r
+              return EFI_INVALID_PARAMETER;\r
+            }\r
+\r
+            //\r
+            // Check current string length is less than maxsize\r
+            //\r
+            if (StrSize ((CHAR16 *) (VarBuffer + Offset)) > Width) {\r
+              return EFI_INVALID_PARAMETER;\r
+            }\r
           }\r
+          break;\r
+        case EFI_IFR_ONE_OF_OPTION_OP:\r
           //\r
-          // Check this var question is in the var storage \r
+          // Opcode Scope is zero. This one of option is not to be checked. \r
           //\r
-          if ((Offset + Width) > IfrVarStore->Size) {\r
-            //\r
-            // This question exceeds the var store size. \r
-            //\r
-            Status = EFI_INVALID_PARAMETER;\r
-            goto Done;\r
+          if (VarBlockData.Scope == 0) {\r
+            break;\r
           }\r
 \r
           //\r
-          // Boolean type, only 1 and 0 is valid.\r
+          // Only check for OneOf and OrderList opcode\r
           //\r
-          if (*(VarBuffer + Offset) > 1) {\r
-            Status = EFI_INVALID_PARAMETER;\r
-            goto Done;            \r
+          IfrOneOfOption = (EFI_IFR_ONE_OF_OPTION *) IfrOpHdr;\r
+          if (VarBlockData.OpCode == EFI_IFR_ONE_OF_OP) {\r
+            //\r
+            // Check current value is the value of one of option.\r
+            //\r
+            ASSERT (IfrOneOfOption->Type <= EFI_IFR_TYPE_NUM_SIZE_64);\r
+            ZeroMem (&TmpValue, sizeof (EFI_IFR_TYPE_VALUE));\r
+            CopyMem (&TmpValue, &IfrOneOfOption->Value, IfrOneOfOption->Header.Length - OFFSET_OF (EFI_IFR_ONE_OF_OPTION, Value));\r
+            if (VarValue == TmpValue.u64) {\r
+              //\r
+              // The value is one of option value.\r
+              // Set OpCode to Zero, don't need check again.\r
+              //\r
+              VarBlockData.OpCode = 0;\r
+            }\r
           }\r
-          \r
           break;\r
-        case EFI_IFR_STRING_OP:\r
+        case EFI_IFR_END_OP:\r
           //\r
-          // Check current string length is less than maxsize\r
+          // Decrease opcode scope for the validated opcode\r
           //\r
+          if (VarBlockData.Scope > 0) {\r
+            VarBlockData.Scope --;\r
+          }\r
 \r
           //\r
-          // CheckBox question is not in IFR Form. This IFR form is not valid\r
+          // OneOf value doesn't belong to one of option value\r
           //\r
-          if (IfrVarStore == NULL) {\r
-            Status = EFI_INVALID_PARAMETER;\r
-            goto Done;\r
+          if ((VarBlockData.Scope == 0) && (VarBlockData.OpCode == EFI_IFR_ONE_OF_OP)) {\r
+            return EFI_INVALID_PARAMETER;\r
           }\r
+          break;\r
+        default:\r
+          //\r
+          // Increase Scope for the validated opcode\r
+          //\r
+          if (VarBlockData.Scope > 0) {\r
+            VarBlockData.Scope = (UINT8) (VarBlockData.Scope + IfrOpHdr->Scope);\r
+          }\r
+          break;\r
+        }\r
+        //\r
+        // Go to the next opcode\r
+        //\r
+        IfrOffset += IfrOpHdr->Length;\r
+      }\r
+      //\r
+      // Only one form is in a package list.\r
+      //\r
+      break;\r
+    }\r
+\r
+    //\r
+    // Go to next package.\r
+    //\r
+    PackageOffset += PackageHeader.Length;\r
+  }\r
+\r
+  return EFI_SUCCESS;\r
+}\r
+\r
+/**\r
+  This internal function parses IFR data to validate current setting.\r
+\r
+  @param ConfigElement         ConfigResp element string contains the current setting.\r
+  @param CurrentBlockArray     Current block array.\r
+  @param VarBuffer             Data buffer for this varstore.\r
+  \r
+  @retval EFI_SUCCESS            The current setting is valid.\r
+  @retval EFI_OUT_OF_RESOURCES   The memory is not enough.\r
+  @retval EFI_INVALID_PARAMETER  The config string or the Hii package is invalid.\r
+**/\r
+EFI_STATUS\r
+GetBlockDataInfo (\r
+  IN  CHAR16                        *ConfigElement,\r
+  OUT IFR_BLOCK_DATA                **CurrentBlockArray,\r
+  OUT UINT8                         **VarBuffer\r
+  )\r
+{\r
+  IFR_BLOCK_DATA               *BlockData;\r
+  IFR_BLOCK_DATA               *NewBlockData;\r
+  EFI_STRING                   StringPtr;\r
+  UINTN                        Length;\r
+  UINT8                        *TmpBuffer;\r
+  UINT16                       Offset;\r
+  UINT16                       Width;\r
+  LIST_ENTRY                   *Link;\r
+  UINTN                        MaxBufferSize;\r
+  EFI_STATUS                   Status;\r
+  IFR_BLOCK_DATA               *BlockArray;\r
+  UINT8                        *DataBuffer;\r
+  \r
+  //\r
+  // Initialize the local variables.\r
+  //\r
+  Status        = EFI_SUCCESS;\r
+  BlockData     = NULL;\r
+  NewBlockData  = NULL;\r
+  TmpBuffer     = NULL;\r
+  BlockArray    = NULL;\r
+  MaxBufferSize = HII_LIB_DEFAULT_VARSTORE_SIZE;\r
+  DataBuffer     = AllocateZeroPool (MaxBufferSize);\r
+  if (DataBuffer == NULL) {\r
+    return EFI_OUT_OF_RESOURCES;\r
+  }\r
+\r
+  //\r
+  // Init BlockArray\r
+  //\r
+  BlockArray = (IFR_BLOCK_DATA *) AllocateZeroPool (sizeof (IFR_BLOCK_DATA));\r
+  if (BlockArray == NULL) {\r
+    Status = EFI_OUT_OF_RESOURCES;\r
+    goto Done;\r
+  }\r
+  InitializeListHead (&BlockArray->Entry);\r
+\r
+  StringPtr = StrStr (ConfigElement, L"&OFFSET=");\r
+  ASSERT (StringPtr != NULL);\r
+\r
+  //\r
+  // Parse each <RequestElement> if exists\r
+  // Only <BlockName> format is supported by this help function.\r
+  // <BlockName> ::= &'OFFSET='<Number>&'WIDTH='<Number>\r
+  //\r
+  while (*StringPtr != 0 && StrnCmp (StringPtr, L"&OFFSET=", StrLen (L"&OFFSET=")) == 0) {\r
+    //\r
+    // Skip the &OFFSET= string\r
+    // \r
+    StringPtr += StrLen (L"&OFFSET=");\r
+\r
+    //\r
+    // Get Offset\r
+    //\r
+    Status = InternalHiiGetValueOfNumber (StringPtr, &TmpBuffer, &Length);\r
+    if (EFI_ERROR (Status)) {\r
+      goto Done;\r
+    }\r
+    Offset = 0;\r
+    CopyMem (\r
+      &Offset,\r
+      TmpBuffer,\r
+      (((Length + 1) / 2) < sizeof (UINT16)) ? ((Length + 1) / 2) : sizeof (UINT16)\r
+      );\r
+    FreePool (TmpBuffer);\r
+    TmpBuffer = NULL;\r
+\r
+    StringPtr += Length;\r
+    if (StrnCmp (StringPtr, L"&WIDTH=", StrLen (L"&WIDTH=")) != 0) {\r
+      Status = EFI_INVALID_PARAMETER;\r
+      goto Done;\r
+    }\r
+    StringPtr += StrLen (L"&WIDTH=");\r
+\r
+    //\r
+    // Get Width\r
+    //\r
+    Status = InternalHiiGetValueOfNumber (StringPtr, &TmpBuffer, &Length);\r
+    if (EFI_ERROR (Status)) {\r
+      goto Done;\r
+    }\r
+    Width = 0;\r
+    CopyMem (\r
+      &Width,\r
+      TmpBuffer,\r
+      (((Length + 1) / 2) < sizeof (UINT16)) ? ((Length + 1) / 2) : sizeof (UINT16)\r
+      );\r
+    FreePool (TmpBuffer);\r
+    TmpBuffer = NULL;\r
+\r
+    StringPtr += Length;\r
+    if (*StringPtr != 0 && *StringPtr != L'&') {\r
+      Status = EFI_INVALID_PARAMETER;\r
+      goto Done;\r
+    }\r
+\r
+    if (StrnCmp (StringPtr, L"&VALUE=", StrLen (L"&VALUE=")) != 0) {\r
+      Status = EFI_INVALID_PARAMETER;\r
+      goto Done;\r
+    }\r
+    StringPtr += StrLen (L"&VALUE=");\r
+\r
+    //\r
+    // Get Value\r
+    //\r
+    Status = InternalHiiGetValueOfNumber (StringPtr, &TmpBuffer, &Length);\r
+    if (EFI_ERROR (Status)) {\r
+      goto Done;\r
+    }\r
+\r
+    StringPtr += Length;\r
+    if (*StringPtr != 0 && *StringPtr != L'&') {\r
+      Status = EFI_INVALID_PARAMETER;\r
+      goto Done;\r
+    }\r
 \r
-          //\r
-          // Check whether this question is for the requested varstore.\r
-          //\r
-          IfrString = (EFI_IFR_STRING *) IfrOpHdr;\r
-          if (IfrString->Question.VarStoreId != IfrVarStore->VarStoreId) {\r
-            break;\r
-          }\r
-          \r
-          //\r
-          // Get Offset/Width by Question header and OneOf Flags\r
-          //\r
-          Offset = IfrString->Question.VarStoreInfo.VarOffset;\r
-          Width  = (UINT16) (IfrString->MaxSize * sizeof (UINT16));\r
-          //\r
-          // Check whether this question is in current block array.\r
-          //\r
-          if (!BlockArrayCheck (CurrentBlockArray, Offset, Width)) {\r
-            //\r
-            // This question is not in the current configuration string. Skip it.\r
-            //\r
-            break;\r
-          }\r
-          //\r
-          // Check this var question is in the var storage \r
-          //\r
-          if ((Offset + Width) > IfrVarStore->Size) {\r
-            //\r
-            // This question exceeds the var store size. \r
-            //\r
-            Status = EFI_INVALID_PARAMETER;\r
-            goto Done;\r
-          }\r
-          \r
-          //\r
-          // Check current string length is less than maxsize\r
-          //\r
-          if (StrSize ((CHAR16 *) (VarBuffer + Offset)) > Width) {\r
-            Status = EFI_INVALID_PARAMETER;\r
-            goto Done;            \r
-          }\r
-          break;\r
-        case EFI_IFR_ONE_OF_OPTION_OP:\r
-          //\r
-          // Opcode Scope is zero. This one of option is not to be checked. \r
-          //\r
-          if (VarBlockData.Scope == 0) {\r
-            break;\r
-          }\r
+    //\r
+    // Check whether VarBuffer is enough\r
+    //\r
+    if ((UINT32)Offset + Width > MaxBufferSize) {\r
+      DataBuffer = ReallocatePool (\r
+                    MaxBufferSize,\r
+                    Offset + Width + HII_LIB_DEFAULT_VARSTORE_SIZE,\r
+                    DataBuffer\r
+                    );\r
+      if (DataBuffer == NULL) {\r
+        Status = EFI_OUT_OF_RESOURCES;\r
+        goto Done;\r
+      }\r
+      MaxBufferSize = Offset + Width + HII_LIB_DEFAULT_VARSTORE_SIZE;\r
+    }\r
 \r
-          //\r
-          // Only check for OneOf and OrderList opcode\r
-          //\r
-          IfrOneOfOption = (EFI_IFR_ONE_OF_OPTION *) IfrOpHdr;\r
-          if (VarBlockData.OpCode == EFI_IFR_ONE_OF_OP) {\r
-            //\r
-            // Check current value is the value of one of option.\r
-            //\r
-            if (VarValue == IfrOneOfOption->Value.u64) {\r
-              //\r
-              // The value is one of option value.\r
-              // Set OpCode to Zero, don't need check again.\r
-              //\r
-              VarBlockData.OpCode = 0;\r
-            }\r
-          }\r
+    //\r
+    // Update the Block with configuration info\r
+    //\r
+    CopyMem (DataBuffer + Offset, TmpBuffer, Width);\r
+    FreePool (TmpBuffer);\r
+    TmpBuffer = NULL;\r
 \r
-          break;\r
-        case EFI_IFR_END_OP:\r
-          //\r
-          // Decrease opcode scope for the validated opcode\r
-          //\r
-          if (VarBlockData.Scope > 0) {\r
-            VarBlockData.Scope --;\r
-          }\r
+    //\r
+    // Set new Block Data\r
+    //\r
+    NewBlockData = (IFR_BLOCK_DATA *) AllocateZeroPool (sizeof (IFR_BLOCK_DATA));\r
+    if (NewBlockData == NULL) {\r
+      Status = EFI_OUT_OF_RESOURCES;\r
+      goto Done;\r
+    }\r
+    NewBlockData->Offset = Offset;\r
+    NewBlockData->Width  = Width;\r
 \r
-          //\r
-          // OneOf value doesn't belong to one of option value. \r
-          //\r
-          if ((VarBlockData.Scope == 0) && (VarBlockData.OpCode == EFI_IFR_ONE_OF_OP)) {\r
-            Status = EFI_INVALID_PARAMETER;\r
-            goto Done;\r
-          }\r
-          break;\r
-        default:\r
-          //\r
-          // Increase Scope for the validated opcode\r
-          //\r
-          if (VarBlockData.Scope > 0) {\r
-            VarBlockData.Scope = (UINT8) (VarBlockData.Scope + IfrOpHdr->Scope);\r
-          }\r
-          break;\r
+    //\r
+    // Insert the new block data into the block data array.\r
+    //\r
+    for (Link = BlockArray->Entry.ForwardLink; Link != &BlockArray->Entry; Link = Link->ForwardLink) {\r
+      BlockData = BASE_CR (Link, IFR_BLOCK_DATA, Entry);\r
+      if (NewBlockData->Offset == BlockData->Offset) {\r
+        if (NewBlockData->Width > BlockData->Width) {\r
+          BlockData->Width = NewBlockData->Width;\r
         }\r
+        FreePool (NewBlockData);\r
+        break;\r
+      } else if (NewBlockData->Offset < BlockData->Offset) {\r
         //\r
-        // Go to the next opcode\r
+        // Insert new block data as the previous one of this link.\r
         //\r
-        IfrOffset += IfrOpHdr->Length;\r
+        InsertTailList (Link, &NewBlockData->Entry);\r
+        break;\r
       }\r
-      //\r
-      // Only one form is in a package list.\r
-      //\r
+    }\r
+\r
+    //\r
+    // Insert new block data into the array tail.\r
+    //\r
+    if (Link == &BlockArray->Entry) {\r
+      InsertTailList (Link, &NewBlockData->Entry);\r
+    }\r
+\r
+    //\r
+    // If '\0', parsing is finished. \r
+    //\r
+    if (*StringPtr == 0) {\r
       break;\r
     }\r
-    \r
     //\r
-    // Go to next package.\r
+    // Go to next ConfigBlock \r
     //\r
-    PackageOffset += PacakgeHeader.Length;      \r
   }\r
 \r
+  //\r
+  // Merge the aligned block data into the single block data.\r
+  //\r
+  Link = BlockArray->Entry.ForwardLink;\r
+  while ((Link != &BlockArray->Entry) && (Link->ForwardLink != &BlockArray->Entry)) {\r
+    BlockData = BASE_CR (Link, IFR_BLOCK_DATA, Entry);\r
+    NewBlockData = BASE_CR (Link->ForwardLink, IFR_BLOCK_DATA, Entry);\r
+    if ((NewBlockData->Offset >= BlockData->Offset) && (NewBlockData->Offset <= (BlockData->Offset + BlockData->Width))) {\r
+      if ((NewBlockData->Offset + NewBlockData->Width) > (BlockData->Offset + BlockData->Width)) {\r
+        BlockData->Width = (UINT16) (NewBlockData->Offset + NewBlockData->Width - BlockData->Offset);\r
+      }\r
+      RemoveEntryList (Link->ForwardLink);\r
+      FreePool (NewBlockData);\r
+      continue;\r
+    }\r
+    Link = Link->ForwardLink;\r
+  }\r
+\r
+  *VarBuffer         = DataBuffer;\r
+  *CurrentBlockArray = BlockArray;\r
+  return EFI_SUCCESS;\r
+\r
 Done:\r
+  if (DataBuffer != NULL) {\r
+    FreePool (DataBuffer);\r
+  }\r
+  \r
+  if (BlockArray != NULL) {\r
+    //\r
+    // Free Link Array CurrentBlockArray\r
+    //\r
+    while (!IsListEmpty (&BlockArray->Entry)) {\r
+      BlockData = BASE_CR (BlockArray->Entry.ForwardLink, IFR_BLOCK_DATA, Entry);\r
+      RemoveEntryList (&BlockData->Entry);\r
+      FreePool (BlockData);\r
+    }\r
+    FreePool (BlockArray);\r
+  }\r
+\r
+  return Status;\r
+}\r
+\r
+/**\r
+  This internal function parses IFR data to validate current setting.\r
+\r
+  @param ConfigResp         ConfigResp string contains the current setting.\r
+  @param HiiPackageList     Point to Hii package list.\r
+  @param PackageListLength  The length of the pacakge.\r
+  @param VarGuid            Guid of the buffer storage.\r
+  @param VarName            Name of the buffer storage.\r
+  @param HiiHandle          The HiiHandle for this package.\r
+  \r
+  @retval EFI_SUCCESS            The current setting is valid.\r
+  @retval EFI_OUT_OF_RESOURCES   The memory is not enough.\r
+  @retval EFI_INVALID_PARAMETER  The config string or the Hii package is invalid.\r
+**/\r
+EFI_STATUS\r
+EFIAPI\r
+InternalHiiValidateCurrentSetting (\r
+  IN EFI_STRING                    ConfigResp,\r
+  IN EFI_HII_PACKAGE_LIST_HEADER   *HiiPackageList,\r
+  IN UINTN                         PackageListLength,\r
+  IN EFI_GUID                      *VarGuid,\r
+  IN CHAR16                        *VarName,\r
+  IN EFI_HII_HANDLE                HiiHandle\r
+  )\r
+{\r
+  CHAR16              *StringPtr;\r
+  EFI_STATUS          Status;\r
+  IFR_BLOCK_DATA      *CurrentBlockArray;\r
+  IFR_BLOCK_DATA      *BlockData;\r
+  UINT8               *VarBuffer;\r
+  BOOLEAN             NameValueType;\r
+\r
+  CurrentBlockArray = NULL;\r
+  VarBuffer         = NULL;\r
+  StringPtr         = NULL;\r
+  Status            = EFI_SUCCESS;\r
+\r
+  //\r
+  // If StringPtr != NULL, get the request elements.\r
+  //\r
+  if (StrStr (ConfigResp, L"&OFFSET=") != NULL) {\r
+    Status = GetBlockDataInfo(ConfigResp, &CurrentBlockArray, &VarBuffer);\r
+    if (EFI_ERROR (Status)) {\r
+      return Status;\r
+    }\r
+    NameValueType = FALSE;\r
+  } else {\r
+    //\r
+    // Skip header part.\r
+    //\r
+    StringPtr = StrStr (ConfigResp, L"PATH=");\r
+    ASSERT (StringPtr != NULL);\r
+\r
+    if (StrStr (StringPtr, L"&") != NULL) {\r
+      NameValueType = TRUE;\r
+    } else {\r
+      //\r
+      // Not found Request element, return success.\r
+      //\r
+      return EFI_SUCCESS;\r
+    }\r
+  }\r
+\r
+  Status = ValidateQuestionFromVfr(\r
+                          HiiPackageList,\r
+                          PackageListLength,\r
+                          VarGuid,\r
+                          VarName,\r
+                          VarBuffer,\r
+                          CurrentBlockArray,\r
+                          ConfigResp,\r
+                          HiiHandle,\r
+                          NameValueType\r
+                          );\r
+\r
   if (VarBuffer != NULL) {\r
     FreePool (VarBuffer);\r
   }\r
@@ -1576,12 +2073,40 @@ Done:
       RemoveEntryList (&BlockData->Entry);\r
       FreePool (BlockData);\r
     }\r
-    FreePool (CurrentBlockArray);    \r
+    FreePool (CurrentBlockArray);\r
   }\r
 \r
   return Status;\r
 }\r
 \r
+/**\r
+  Check whether the ConfigRequest string has the request elements.\r
+  For EFI_HII_VARSTORE_BUFFER type, the request has "&OFFSET=****&WIDTH=****..." format.\r
+  For EFI_HII_VARSTORE_NAME_VALUE type, the request has "&NAME1**&NAME2..." format.\r
+\r
+  @param  ConfigRequest      The input config request string.\r
+\r
+  @retval  TRUE              The input include config request elements.\r
+  @retval  FALSE             The input string not includes.\r
+                                 \r
+**/\r
+BOOLEAN\r
+GetElementsFromRequest (\r
+  IN EFI_STRING    ConfigRequest\r
+  )\r
+{\r
+  EFI_STRING   TmpRequest;\r
+\r
+  TmpRequest = StrStr (ConfigRequest, L"PATH=");\r
+  ASSERT (TmpRequest != NULL);\r
+\r
+  if ((StrStr (TmpRequest, L"&OFFSET=") != NULL) || (StrStr (TmpRequest, L"&") != NULL)) {\r
+    return TRUE;\r
+  }\r
+\r
+  return FALSE;\r
+}\r
+\r
 /**\r
   This function parses the input ConfigRequest string and its matched IFR code\r
   string for setting default value and validating current setting.\r
@@ -1603,7 +2128,7 @@ Done:
   @param DefaultId  Specifies the type of defaults to retrieve only for setting default action.\r
   @param ActionType Action supports setting defaults and validate current setting.\r
   \r
-  @retval TURE    Action runs successfully.\r
+  @retval TRUE    Action runs successfully.\r
   @retval FALSE   Action is not valid or Action can't be executed successfully..\r
 **/\r
 BOOLEAN\r
@@ -1631,6 +2156,7 @@ InternalHiiIfrValueAction (
 \r
   EFI_HII_PACKAGE_LIST_HEADER  *HiiPackageList;\r
   UINTN                        PackageListLength;\r
+  UINTN                        MaxLen;\r
   EFI_DEVICE_PATH_PROTOCOL     *DevicePath;\r
   EFI_DEVICE_PATH_PROTOCOL     *TempDevicePath;\r
 \r
@@ -1761,7 +2287,7 @@ InternalHiiIfrValueAction (
       // Its default value and validating can't execute by parsing IFR data.\r
       // Directly jump into the next ConfigAltResp string for another pair Guid, Name, and Path.   \r
       //\r
-           Status = EFI_SUCCESS;\r
+      Status = EFI_SUCCESS;\r
       goto NextConfigAltResp;\r
     }\r
 \r
@@ -1818,7 +2344,7 @@ InternalHiiIfrValueAction (
     //\r
     // Only the ConfigHdr is found. Not any block data is found. No data is required to be validated and set.\r
     //\r
-    if (StrStr (ConfigResp, L"&OFFSET=") == NULL) {\r
+    if (!GetElementsFromRequest (ConfigResp)) {\r
       goto NextConfigAltResp;\r
     }\r
     \r
@@ -1835,7 +2361,7 @@ InternalHiiIfrValueAction (
       //\r
       // Current Setting is in ConfigResp, will be set into buffer, then check it again.\r
       //\r
-      Status = InternalHiiValidateCurrentSetting (ConfigResp, HiiPackageList, PackageListLength, VarGuid, VarName);\r
+      Status = InternalHiiValidateCurrentSetting (ConfigResp, HiiPackageList, PackageListLength, VarGuid, VarName, HiiHandle);\r
     }\r
 \r
     if (EFI_ERROR (Status)) {\r
@@ -1851,10 +2377,10 @@ NextConfigAltResp:
       HiiPackageList = NULL;\r
     }\r
     \r
-       if (ConfigResp != NULL) {\r
-         FreePool (ConfigResp);\r
-         ConfigResp = NULL;\r
-       }\r
+    if (ConfigResp != NULL) {\r
+      FreePool (ConfigResp);\r
+      ConfigResp = NULL;\r
+    }\r
 \r
     //\r
     // Free the allocated buffer.\r
@@ -1886,14 +2412,15 @@ NextConfigAltResp:
     // Construct ConfigAltHdr string  "&<ConfigHdr>&ALTCFG=\0" \r
     //                               | 1 | StrLen (ConfigHdr) | 8 | 1 |\r
     //\r
-    ConfigAltHdr = AllocateZeroPool ((1 + StringPtr - StringHdr + 8 + 1) * sizeof (CHAR16));\r
+    MaxLen = 1 + StringPtr - StringHdr + 8 + 1;\r
+    ConfigAltHdr = AllocateZeroPool ( MaxLen * sizeof (CHAR16));\r
     if (ConfigAltHdr == NULL) {\r
       Status = EFI_OUT_OF_RESOURCES;\r
       goto Done;\r
     }\r
-    StrCpy (ConfigAltHdr, L"&");\r
-    StrnCat (ConfigAltHdr, StringHdr, StringPtr - StringHdr);\r
-    StrCat (ConfigAltHdr, L"&ALTCFG=");\r
+    StrCpyS (ConfigAltHdr, MaxLen, L"&");\r
+    StrnCatS (ConfigAltHdr, MaxLen, StringHdr, StringPtr - StringHdr);\r
+    StrCatS (ConfigAltHdr, MaxLen, L"&ALTCFG=");\r
     \r
     //\r
     // Skip all AltResp (AltConfigHdr ConfigBody) for the same ConfigHdr\r
@@ -1995,7 +2522,7 @@ HiiValidateSettings (
                     entirety of the current HII database will be reset.\r
   @param DefaultId  Specifies the type of defaults to retrieve.\r
   \r
-  @retval TURE    The default value is set successfully.\r
+  @retval TRUE    The default value is set successfully.\r
   @retval FALSE   The default value can't be found and set.\r
 **/\r
 BOOLEAN\r
@@ -2628,7 +3155,7 @@ HiiCreateOneOfOptionOpCode (
   OpCode.Type   = Type;\r
   CopyMem (&OpCode.Value, &Value, mHiiDefaultTypeToWidth[Type]);\r
 \r
-  return InternalHiiCreateOpCode (OpCodeHandle, &OpCode, EFI_IFR_ONE_OF_OPTION_OP, sizeof (OpCode));\r
+  return InternalHiiCreateOpCode (OpCodeHandle, &OpCode, EFI_IFR_ONE_OF_OPTION_OP, OFFSET_OF(EFI_IFR_ONE_OF_OPTION, Value) + mHiiDefaultTypeToWidth[Type]);\r
 }\r
 \r
 /**\r
@@ -2854,6 +3381,82 @@ HiiCreateGotoOpCode (
   return InternalHiiCreateOpCode (OpCodeHandle, &OpCode, EFI_IFR_REF_OP, sizeof (OpCode));\r
 }\r
 \r
+/**\r
+  Create EFI_IFR_REF_OP, EFI_IFR_REF2_OP, EFI_IFR_REF3_OP and EFI_IFR_REF4_OP opcode.\r
+\r
+  When RefDevicePath is not zero, EFI_IFR_REF4 opcode will be created. \r
+  When RefDevicePath is zero and RefFormSetId is not NULL, EFI_IFR_REF3 opcode will be created.\r
+  When RefDevicePath is zero, RefFormSetId is NULL and RefQuestionId is not zero, EFI_IFR_REF2 opcode will be created.\r
+  When RefDevicePath is zero, RefFormSetId is NULL and RefQuestionId is zero, EFI_IFR_REF opcode will be created.\r
+\r
+  If OpCodeHandle is NULL, then ASSERT().\r
+  If any reserved bits are set in QuestionFlags, then ASSERT().\r
+\r
+  @param[in]  OpCodeHandle   The handle to the buffer of opcodes.\r
+  @param[in]  RefFormId      The Destination Form ID.\r
+  @param[in]  Prompt         The string ID for Prompt.\r
+  @param[in]  Help           The string ID for Help.\r
+  @param[in]  QuestionFlags  The flags in Question Header\r
+  @param[in]  QuestionId     Question ID.\r
+  @param[in]  RefQuestionId  The question on the form to which this link is referring. \r
+                             If its value is zero, then the link refers to the top of the form.\r
+  @param[in]  RefFormSetId   The form set to which this link is referring. If its value is NULL, and RefDevicePath is \r
+                             zero, then the link is to the current form set.\r
+  @param[in]  RefDevicePath  The string identifier that specifies the string containing the text representation of \r
+                             the device path to which the form set containing the form specified by FormId.\r
+                             If its value is zero, then the link refers to the current page.\r
+\r
+  @retval NULL   There is not enough space left in Buffer to add the opcode.\r
+  @retval Other  A pointer to the created opcode.\r
+\r
+**/\r
+UINT8 *\r
+EFIAPI\r
+HiiCreateGotoExOpCode (\r
+  IN VOID             *OpCodeHandle,\r
+  IN EFI_FORM_ID      RefFormId,\r
+  IN EFI_STRING_ID    Prompt,\r
+  IN EFI_STRING_ID    Help,\r
+  IN UINT8            QuestionFlags,\r
+  IN EFI_QUESTION_ID  QuestionId,\r
+  IN EFI_QUESTION_ID  RefQuestionId,\r
+  IN EFI_GUID         *RefFormSetId,    OPTIONAL\r
+  IN EFI_STRING_ID    RefDevicePath\r
+  )\r
+{\r
+  EFI_IFR_REF4  OpCode;\r
+  UINTN         OpCodeSize;\r
+\r
+  ASSERT ((QuestionFlags & (~(EFI_IFR_FLAG_READ_ONLY | EFI_IFR_FLAG_CALLBACK | EFI_IFR_FLAG_RESET_REQUIRED))) == 0);\r
+\r
+  ZeroMem (&OpCode, sizeof (OpCode));\r
+  OpCode.Question.Header.Prompt = Prompt;\r
+  OpCode.Question.Header.Help   = Help;\r
+  OpCode.Question.QuestionId    = QuestionId;\r
+  OpCode.Question.Flags         = QuestionFlags;\r
+  OpCode.FormId                 = RefFormId;\r
+  OpCode.QuestionId             = RefQuestionId;\r
+  OpCode.DevicePath             = RefDevicePath;\r
+  if (RefFormSetId != NULL) {\r
+    CopyMem (&OpCode.FormSetId, RefFormSetId, sizeof (OpCode.FormSetId));\r
+  }\r
+\r
+  //\r
+  // Cacluate OpCodeSize based on the input Ref value.\r
+  // Try to use the small OpCode to save size.\r
+  //\r
+  OpCodeSize = sizeof (EFI_IFR_REF);\r
+  if (RefDevicePath != 0) {\r
+    OpCodeSize = sizeof (EFI_IFR_REF4);\r
+  } else if (RefFormSetId != NULL) {\r
+    OpCodeSize = sizeof (EFI_IFR_REF3);\r
+  } else if (RefQuestionId != 0) {\r
+    OpCodeSize = sizeof (EFI_IFR_REF2);\r
+  }\r
+\r
+  return InternalHiiCreateOpCode (OpCodeHandle, &OpCode, EFI_IFR_REF_OP, OpCodeSize);\r
+}\r
+\r
 /**\r
   Create EFI_IFR_CHECKBOX_OP opcode.\r
 \r
@@ -2864,7 +3467,8 @@ HiiCreateGotoOpCode (
   @param[in]  OpCodeHandle          Handle to the buffer of opcodes.\r
   @param[in]  QuestionId            Question ID\r
   @param[in]  VarStoreId            Storage ID\r
-  @param[in]  VarOffset             Offset in Storage\r
+  @param[in]  VarOffset             Offset in Storage or String ID of the name (VarName)\r
+                                    for this name/value pair.\r
   @param[in]  Prompt                String ID for Prompt\r
   @param[in]  Help                  String ID for Help\r
   @param[in]  QuestionFlags         Flags in Question Header\r
@@ -2925,7 +3529,8 @@ HiiCreateCheckBoxOpCode (
   @param[in]  OpCodeHandle          Handle to the buffer of opcodes.\r
   @param[in]  QuestionId            Question ID\r
   @param[in]  VarStoreId            Storage ID\r
-  @param[in]  VarOffset             Offset in Storage\r
+  @param[in]  VarOffset             Offset in Storage or String ID of the name (VarName)\r
+                                    for this name/value pair.\r
   @param[in]  Prompt                String ID for Prompt\r
   @param[in]  Help                  String ID for Help\r
   @param[in]  QuestionFlags         Flags in Question Header\r
@@ -3026,7 +3631,8 @@ HiiCreateNumericOpCode (
   @param[in]  OpCodeHandle          Handle to the buffer of opcodes.\r
   @param[in]  QuestionId            Question ID\r
   @param[in]  VarStoreId            Storage ID\r
-  @param[in]  VarOffset             Offset in Storage\r
+  @param[in]  VarOffset             Offset in Storage or String ID of the name (VarName)\r
+                                    for this name/value pair.\r
   @param[in]  Prompt                String ID for Prompt\r
   @param[in]  Help                  String ID for Help\r
   @param[in]  QuestionFlags         Flags in Question Header\r
@@ -3093,7 +3699,8 @@ HiiCreateStringOpCode (
   @param[in]  OpCodeHandle          Handle to the buffer of opcodes.\r
   @param[in]  QuestionId            Question ID\r
   @param[in]  VarStoreId            Storage ID\r
-  @param[in]  VarOffset             Offset in Storage\r
+  @param[in]  VarOffset             Offset in Storage or String ID of the name (VarName)\r
+                                    for this name/value pair.\r
   @param[in]  Prompt                String ID for Prompt\r
   @param[in]  Help                  String ID for Help\r
   @param[in]  QuestionFlags         Flags in Question Header\r
@@ -3160,7 +3767,8 @@ HiiCreateOneOfOpCode (
   @param[in]  OpCodeHandle          Handle to the buffer of opcodes.\r
   @param[in]  QuestionId            Question ID\r
   @param[in]  VarStoreId            Storage ID\r
-  @param[in]  VarOffset             Offset in Storage\r
+  @param[in]  VarOffset             Offset in Storage or String ID of the name (VarName)\r
+                                    for this name/value pair.\r
   @param[in]  Prompt                String ID for Prompt\r
   @param[in]  Help                  String ID for Help\r
   @param[in]  QuestionFlags         Flags in Question Header\r
@@ -3262,7 +3870,8 @@ HiiCreateTextOpCode (
   @param[in]  QuestionId            Question ID\r
   @param[in]  VarStoreId            Storage ID, optional. If DateFlags is not\r
                                     QF_DATE_STORAGE_NORMAL, this parameter is ignored.\r
-  @param[in]  VarOffset             Offset in Storage, optional. If DateFlags is not\r
+  @param[in]  VarOffset             Offset in Storage or String ID of the name (VarName)\r
+                                    for this name/value pair, optional. If DateFlags is not\r
                                     QF_DATE_STORAGE_NORMAL, this parameter is ignored.\r
   @param[in]  Prompt                String ID for Prompt\r
   @param[in]  Help                  String ID for Help\r
@@ -3326,7 +3935,8 @@ HiiCreateDateOpCode (
   @param[in]  QuestionId            Question ID\r
   @param[in]  VarStoreId            Storage ID, optional. If TimeFlags is not\r
                                     QF_TIME_STORAGE_NORMAL, this parameter is ignored.\r
-  @param[in]  VarOffset             Offset in Storage, optional. If TimeFlags is not\r
+  @param[in]  VarOffset             Offset in Storage or String ID of the name (VarName)\r
+                                    for this name/value pair, optional. If TimeFlags is not\r
                                     QF_TIME_STORAGE_NORMAL, this parameter is ignored.\r
   @param[in]  Prompt                String ID for Prompt\r
   @param[in]  Help                  String ID for Help\r
@@ -3617,7 +4227,7 @@ HiiUpdateForm (
   UINTN                        BufferSize;\r
   UINT8                        *UpdateBufferPos;\r
   EFI_HII_PACKAGE_HEADER       *Package;\r
-  EFI_HII_PACKAGE_HEADER       *TempPacakge;\r
+  EFI_HII_PACKAGE_HEADER       *TempPackage;\r
   EFI_HII_PACKAGE_HEADER       PackageHeader;\r
   BOOLEAN                      Updated;\r
   HII_LIB_OPCODE_BUFFER        *OpCodeBufferStart;\r
@@ -3629,7 +4239,7 @@ HiiUpdateForm (
   ASSERT (HiiHandle != NULL);\r
   ASSERT (StartOpCodeHandle != NULL);\r
   UpdatePackageList = NULL;\r
-  TempPacakge       = NULL;\r
+  TempPackage       = NULL;\r
   HiiPackageList    = NULL;\r
   \r
   //\r
@@ -3675,8 +4285,8 @@ HiiUpdateForm (
   //\r
   // Allocate temp buffer to store the temp updated package buffer\r
   //\r
-  TempPacakge = AllocateZeroPool (BufferSize);\r
-  if (TempPacakge == NULL) {\r
+  TempPackage = AllocateZeroPool (BufferSize);\r
+  if (TempPackage == NULL) {\r
     Status = EFI_OUT_OF_RESOURCES;\r
     goto Finish;\r
   }\r
@@ -3704,7 +4314,7 @@ HiiUpdateForm (
       //\r
       // Check this package is the matched package.\r
       //\r
-      Status = InternalHiiUpdateFormPackageData (FormSetGuid, FormId, Package, OpCodeBufferStart, OpCodeBufferEnd, TempPacakge);\r
+      Status = InternalHiiUpdateFormPackageData (FormSetGuid, FormId, Package, OpCodeBufferStart, OpCodeBufferEnd, TempPackage);\r
       //\r
       // The matched package is found. Its package buffer will be updated by the input new data.\r
       //\r
@@ -3716,7 +4326,7 @@ HiiUpdateForm (
         //\r
         // Add updated package buffer\r
         //\r
-        Package = TempPacakge;\r
+        Package = TempPackage;\r
       }\r
     }\r
 \r
@@ -3755,8 +4365,8 @@ Finish:
     FreePool (UpdatePackageList);\r
   }\r
   \r
-  if (TempPacakge != NULL) {\r
-    FreePool (TempPacakge);\r
+  if (TempPackage != NULL) {\r
+    FreePool (TempPackage);\r
   }\r
 \r
   return Status; \r