]> git.proxmox.com Git - mirror_edk2.git/blobdiff - MdeModulePkg/Universal/SetupBrowserDxe/IfrParse.c
Update the logic, only check the value change status for user input action, not detec...
[mirror_edk2.git] / MdeModulePkg / Universal / SetupBrowserDxe / IfrParse.c
index 7493a943027ebea1eda4c6e7eb02ff7a96341d27..41f89ea7f61eabe76fc24654685268225da527ca 100644 (file)
@@ -1,7 +1,7 @@
 /** @file\r
 Parser for IFR binary encoding.\r
 \r
-Copyright (c) 2007 - 2012, Intel Corporation. All rights reserved.<BR>\r
+Copyright (c) 2007 - 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
@@ -16,8 +16,8 @@ WITHOUT WARRANTIES OR REPRESENTATIONS OF ANY KIND, EITHER EXPRESS OR IMPLIED.
 \r
 UINT16           mStatementIndex;\r
 UINT16           mExpressionOpCodeIndex;\r
-\r
-BOOLEAN          mInScopeSubtitle;\r
+EFI_QUESTION_ID  mUsedQuestionId;\r
+extern LIST_ENTRY      gBrowserStorageList;\r
 /**\r
   Initialize Statement header members.\r
 \r
@@ -41,9 +41,9 @@ CreateStatement (
 \r
   if (Form == NULL) {\r
     //\r
-    // We are currently not in a Form Scope, so just skip this Statement\r
+    // Only guid op may out side the form level.\r
     //\r
-    return NULL;\r
+    ASSERT (((EFI_IFR_OP_HEADER *) OpCodeData)->OpCode == EFI_IFR_GUID_OP);\r
   }\r
 \r
   Statement = &FormSet->StatementBuffer[mStatementIndex];\r
@@ -53,10 +53,12 @@ CreateStatement (
   InitializeListHead (&Statement->OptionListHead);\r
   InitializeListHead (&Statement->InconsistentListHead);\r
   InitializeListHead (&Statement->NoSubmitListHead);\r
+  InitializeListHead (&Statement->WarningListHead);\r
 \r
   Statement->Signature = FORM_BROWSER_STATEMENT_SIGNATURE;\r
 \r
   Statement->Operand = ((EFI_IFR_OP_HEADER *) OpCodeData)->OpCode;\r
+  Statement->OpCode  = (EFI_IFR_OP_HEADER *) OpCodeData;\r
 \r
   StatementHdr = (EFI_IFR_STATEMENT_HEADER *) (OpCodeData + sizeof (EFI_IFR_OP_HEADER));\r
   CopyMem (&Statement->Prompt, &StatementHdr->Prompt, sizeof (EFI_STRING_ID));\r
@@ -76,13 +78,14 @@ CreateStatement (
     CopyMem (Statement->Expression->Expression, GetConditionalExpressionList(ExpressStatement), (UINTN) (sizeof (FORM_EXPRESSION *) * ConditionalExprCount));\r
   }\r
 \r
-  Statement->InSubtitle = mInScopeSubtitle;\r
-\r
   //\r
   // Insert this Statement into current Form\r
   //\r
-  InsertTailList (&Form->StatementListHead, &Statement->Link);\r
-\r
+  if (Form == NULL) {\r
+    InsertTailList (&FormSet->StatementListOSF, &Statement->Link);\r
+  } else {\r
+    InsertTailList (&Form->StatementListHead, &Statement->Link);\r
+  }\r
   return Statement;\r
 }\r
 \r
@@ -179,6 +182,7 @@ CreateQuestion (
   FORMSET_STORAGE          *Storage;\r
   NAME_VALUE_NODE          *NameValueNode;\r
   EFI_STATUS               Status;\r
+  BOOLEAN                  Find;\r
 \r
   Statement = CreateStatement (OpCodeData, FormSet, Form);\r
   if (Statement == NULL) {\r
@@ -220,7 +224,7 @@ CreateQuestion (
     Storage = FORMSET_STORAGE_FROM_LINK (Link);\r
 \r
     if (Storage->VarStoreId == Statement->VarStoreId) {\r
-      Statement->Storage = Storage;\r
+      Statement->Storage = Storage->BrowserStorage;\r
       break;\r
     }\r
 \r
@@ -238,19 +242,39 @@ CreateQuestion (
 \r
     if (Statement->Storage->Type == EFI_HII_VARSTORE_NAME_VALUE) {\r
       //\r
-      // Insert to Name/Value varstore list\r
+      // Check whether old string node already exist.\r
       //\r
-      NameValueNode = AllocateZeroPool (sizeof (NAME_VALUE_NODE));\r
-      ASSERT (NameValueNode != NULL);\r
-      NameValueNode->Signature = NAME_VALUE_NODE_SIGNATURE;\r
-      NameValueNode->Name = AllocateCopyPool (StrSize (Statement->VariableName), Statement->VariableName);\r
-      ASSERT (NameValueNode->Name != NULL);\r
-      NameValueNode->Value = AllocateZeroPool (0x10);\r
-      ASSERT (NameValueNode->Value != NULL);\r
-      NameValueNode->EditValue = AllocateZeroPool (0x10);\r
-      ASSERT (NameValueNode->EditValue != NULL);\r
+      Find = FALSE;\r
+      if (!IsListEmpty(&Statement->Storage->NameValueListHead)) {  \r
+        Link = GetFirstNode (&Statement->Storage->NameValueListHead);\r
+        while (!IsNull (&Statement->Storage->NameValueListHead, Link)) {\r
+          NameValueNode = NAME_VALUE_NODE_FROM_LINK (Link);\r
+\r
+          if (StrCmp (Statement->VariableName, NameValueNode->Name) == 0) {\r
+            Find = TRUE;\r
+            break;\r
+          }\r
 \r
-      InsertTailList (&Statement->Storage->NameValueListHead, &NameValueNode->Link);\r
+          Link = GetNextNode (&Statement->Storage->NameValueListHead, Link);\r
+        }\r
+      }\r
+\r
+      if (!Find) {\r
+        //\r
+        // Insert to Name/Value varstore list\r
+        //\r
+        NameValueNode = AllocateZeroPool (sizeof (NAME_VALUE_NODE));\r
+        ASSERT (NameValueNode != NULL);\r
+        NameValueNode->Signature = NAME_VALUE_NODE_SIGNATURE;\r
+        NameValueNode->Name = AllocateCopyPool (StrSize (Statement->VariableName), Statement->VariableName);\r
+        ASSERT (NameValueNode->Name != NULL);\r
+        NameValueNode->Value = AllocateZeroPool (0x10);\r
+        ASSERT (NameValueNode->Value != NULL);\r
+        NameValueNode->EditValue = AllocateZeroPool (0x10);\r
+        ASSERT (NameValueNode->EditValue != NULL);\r
+\r
+        InsertTailList (&Statement->Storage->NameValueListHead, &NameValueNode->Link);\r
+      }\r
     }\r
   }\r
 \r
@@ -281,32 +305,6 @@ CreateExpression (
   return Expression;\r
 }\r
 \r
-\r
-/**\r
-  Allocate a FORMSET_STORAGE data structure and insert to FormSet Storage List.\r
-\r
-  @param  FormSet                Pointer of the current FormSet\r
-\r
-  @return Pointer to a FORMSET_STORAGE data structure.\r
-\r
-**/\r
-FORMSET_STORAGE *\r
-CreateStorage (\r
-  IN FORM_BROWSER_FORMSET  *FormSet\r
-  )\r
-{\r
-  FORMSET_STORAGE  *Storage;\r
-\r
-  Storage = AllocateZeroPool (sizeof (FORMSET_STORAGE));\r
-  ASSERT (Storage != NULL);\r
-  Storage->Signature = FORMSET_STORAGE_SIGNATURE;\r
-  InitializeListHead (&Storage->NameValueListHead);\r
-  InsertTailList (&FormSet->StorageListHead, &Storage->Link);\r
-\r
-  return Storage;\r
-}\r
-\r
-\r
 /**\r
   Create ConfigHdr string for a storage.\r
 \r
@@ -319,7 +317,7 @@ CreateStorage (
 EFI_STATUS\r
 InitializeConfigHdr (\r
   IN FORM_BROWSER_FORMSET  *FormSet,\r
-  IN OUT FORMSET_STORAGE   *Storage\r
+  IN OUT BROWSER_STORAGE   *Storage\r
   )\r
 {\r
   CHAR16      *Name;\r
@@ -341,12 +339,200 @@ InitializeConfigHdr (
     return EFI_NOT_FOUND;\r
   }\r
 \r
-  Storage->ConfigRequest = AllocateCopyPool (StrSize (Storage->ConfigHdr), Storage->ConfigHdr);\r
-  Storage->SpareStrLen = 0;\r
-\r
   return EFI_SUCCESS;\r
 }\r
 \r
+/**\r
+  Find the global storage link base on the input storate type, name and guid.\r
+\r
+  For EFI_HII_VARSTORE_EFI_VARIABLE and EFI_HII_VARSTORE_EFI_VARIABLE_BUFFER,\r
+  same guid + name = same storage\r
+\r
+  For EFI_HII_VARSTORE_NAME_VALUE:\r
+  same guid + HiiHandle = same storage\r
+\r
+  For EFI_HII_VARSTORE_BUFFER:\r
+  same guid + name + HiiHandle = same storage\r
+\r
+  @param  StorageType                Storage type.\r
+  @param  StorageGuid                Storage guid.\r
+  @param  StorageName                Storage Name.\r
+  @param  HiiHandle                  HiiHandle for this varstore.\r
+\r
+  @return Pointer to a GLOBAL_STORAGE data structure.\r
+\r
+**/\r
+BROWSER_STORAGE *\r
+FindStorageInList (\r
+  IN UINT8                 StorageType,\r
+  IN EFI_GUID              *StorageGuid,\r
+  IN CHAR16                *StorageName,\r
+  IN EFI_HII_HANDLE        HiiHandle\r
+  )\r
+{\r
+  LIST_ENTRY       *Link;\r
+  BROWSER_STORAGE  *BrowserStorage;\r
+\r
+  Link  = GetFirstNode (&gBrowserStorageList);\r
+  while (!IsNull (&gBrowserStorageList, Link)) {\r
+    BrowserStorage = BROWSER_STORAGE_FROM_LINK (Link);\r
+    Link = GetNextNode (&gBrowserStorageList, Link);\r
+\r
+    if ((BrowserStorage->Type == StorageType) && CompareGuid (&BrowserStorage->Guid, StorageGuid)) {\r
+      if (StorageType == EFI_HII_VARSTORE_NAME_VALUE) {\r
+        if (BrowserStorage->HiiHandle == HiiHandle) {\r
+          return BrowserStorage;\r
+        }\r
+\r
+        continue;\r
+      }\r
+\r
+      ASSERT (StorageName != NULL);\r
+      if (StrCmp (BrowserStorage->Name, StorageName) == 0) {\r
+        if (StorageType == EFI_HII_VARSTORE_EFI_VARIABLE || StorageType == EFI_HII_VARSTORE_EFI_VARIABLE_BUFFER) {\r
+          return BrowserStorage;\r
+        } else if (StorageType == EFI_HII_VARSTORE_BUFFER && BrowserStorage->HiiHandle == HiiHandle) {\r
+          return BrowserStorage;\r
+        }\r
+      }\r
+    }\r
+  }\r
+\r
+  return NULL;\r
+}\r
+\r
+/**\r
+  Intialize the Global Storage.\r
+\r
+  @param  BrowserStorage              Pointer to the global storage.\r
+  @param  StorageType                Storage type.\r
+  @param  OpCodeData                 Binary data for this opcode.\r
+\r
+**/\r
+VOID\r
+IntializeBrowserStorage (\r
+  IN BROWSER_STORAGE       *BrowserStorage,\r
+  IN UINT8                 StorageType,\r
+  IN UINT8                 *OpCodeData\r
+  )\r
+{\r
+  switch (StorageType) {\r
+    case EFI_HII_VARSTORE_BUFFER:\r
+      CopyMem (&BrowserStorage->Guid, &((EFI_IFR_VARSTORE *) OpCodeData)->Guid, sizeof (EFI_GUID));\r
+      CopyMem (&BrowserStorage->Size, &((EFI_IFR_VARSTORE *) OpCodeData)->Size, sizeof (UINT16));\r
+\r
+      BrowserStorage->Buffer     = AllocateZeroPool (BrowserStorage->Size);\r
+      BrowserStorage->EditBuffer = AllocateZeroPool (BrowserStorage->Size);\r
+      break;\r
+\r
+    case EFI_HII_VARSTORE_EFI_VARIABLE:\r
+    case EFI_HII_VARSTORE_EFI_VARIABLE_BUFFER:\r
+      CopyMem (&BrowserStorage->Guid,       &((EFI_IFR_VARSTORE_EFI *) OpCodeData)->Guid,       sizeof (EFI_GUID));\r
+      CopyMem (&BrowserStorage->Attributes, &((EFI_IFR_VARSTORE_EFI *) OpCodeData)->Attributes, sizeof (UINT32));\r
+      CopyMem (&BrowserStorage->Size,       &((EFI_IFR_VARSTORE_EFI *) OpCodeData)->Size,       sizeof (UINT16));\r
+\r
+      if (StorageType ==  EFI_HII_VARSTORE_EFI_VARIABLE_BUFFER) {\r
+        BrowserStorage->Buffer     = AllocateZeroPool (BrowserStorage->Size);\r
+        BrowserStorage->EditBuffer = AllocateZeroPool (BrowserStorage->Size);\r
+      }\r
+      break;\r
+\r
+    case EFI_HII_VARSTORE_NAME_VALUE:\r
+      CopyMem (&BrowserStorage->Guid, &((EFI_IFR_VARSTORE_NAME_VALUE *) OpCodeData)->Guid, sizeof (EFI_GUID));\r
+\r
+      InitializeListHead (&BrowserStorage->NameValueListHead);\r
+      break;\r
+\r
+    default:\r
+      break;\r
+  }\r
+}\r
+\r
+/**\r
+  Allocate a FORMSET_STORAGE data structure and insert to FormSet Storage List.\r
+\r
+  @param  FormSet                    Pointer of the current FormSet\r
+  @param  StorageType                Storage type.\r
+  @param  OpCodeData                 Binary data for this opcode.\r
+\r
+  @return Pointer to a FORMSET_STORAGE data structure.\r
+\r
+**/\r
+FORMSET_STORAGE *\r
+CreateStorage (\r
+  IN FORM_BROWSER_FORMSET  *FormSet,\r
+  IN UINT8                 StorageType,\r
+  IN UINT8                 *OpCodeData\r
+  )\r
+{\r
+  FORMSET_STORAGE         *Storage;\r
+  CHAR16                  *UnicodeString;\r
+  UINT16                  Index;\r
+  BROWSER_STORAGE         *BrowserStorage;\r
+  EFI_GUID                *StorageGuid;\r
+  CHAR8                   *StorageName;\r
+\r
+  UnicodeString = NULL;\r
+  StorageName   = NULL;\r
+  switch (StorageType) {\r
+    case EFI_HII_VARSTORE_BUFFER:\r
+      StorageGuid = (EFI_GUID *) (CHAR8*) &((EFI_IFR_VARSTORE *) OpCodeData)->Guid;\r
+      StorageName = (CHAR8 *) ((EFI_IFR_VARSTORE *) OpCodeData)->Name;\r
+      break;\r
+\r
+    case EFI_HII_VARSTORE_EFI_VARIABLE:\r
+    case EFI_HII_VARSTORE_EFI_VARIABLE_BUFFER:\r
+      StorageGuid = (EFI_GUID *) (CHAR8*) &((EFI_IFR_VARSTORE_EFI *) OpCodeData)->Guid;\r
+      StorageName = (CHAR8 *) ((EFI_IFR_VARSTORE_EFI *) OpCodeData)->Name;\r
+      break;\r
+\r
+    default:\r
+      ASSERT (StorageType == EFI_HII_VARSTORE_NAME_VALUE);\r
+      StorageGuid = &((EFI_IFR_VARSTORE_NAME_VALUE *) OpCodeData)->Guid;\r
+      break;\r
+  }\r
+\r
+  if (StorageType != EFI_HII_VARSTORE_NAME_VALUE) {\r
+    ASSERT (StorageName != NULL);\r
+\r
+    UnicodeString = AllocateZeroPool (AsciiStrSize (StorageName) * 2);\r
+    ASSERT (UnicodeString != NULL);\r
+    for (Index = 0; StorageName[Index] != 0; Index++) {\r
+      UnicodeString[Index] = (CHAR16) StorageName[Index];\r
+    }\r
+  }\r
+\r
+  Storage = AllocateZeroPool (sizeof (FORMSET_STORAGE));\r
+  ASSERT (Storage != NULL);\r
+  Storage->Signature = FORMSET_STORAGE_SIGNATURE;\r
+  InsertTailList (&FormSet->StorageListHead, &Storage->Link);\r
+\r
+  BrowserStorage = FindStorageInList(StorageType, StorageGuid, UnicodeString, FormSet->HiiHandle);\r
+  if (BrowserStorage == NULL) {\r
+    BrowserStorage = AllocateZeroPool (sizeof (BROWSER_STORAGE));\r
+    ASSERT (BrowserStorage != NULL);\r
+\r
+    BrowserStorage->Signature = BROWSER_STORAGE_SIGNATURE;\r
+    InsertTailList (&gBrowserStorageList, &BrowserStorage->Link);\r
+\r
+    IntializeBrowserStorage (BrowserStorage, StorageType, OpCodeData);\r
+    BrowserStorage->Type = StorageType;\r
+    if (StorageType != EFI_HII_VARSTORE_NAME_VALUE) {\r
+      BrowserStorage->Name = UnicodeString;\r
+    }\r
+\r
+    BrowserStorage->HiiHandle = FormSet->HiiHandle;\r
+    InitializeConfigHdr (FormSet, BrowserStorage);\r
+\r
+    BrowserStorage->Initialized = FALSE;\r
+  }\r
+\r
+  Storage->BrowserStorage = BrowserStorage;\r
+  Storage->ConfigRequest = AllocateCopyPool (StrSize (BrowserStorage->ConfigHdr), BrowserStorage->ConfigHdr);\r
+  Storage->SpareStrLen = 0;\r
+\r
+  return Storage;\r
+}\r
 \r
 /**\r
   Initialize Request Element of a Question. <RequestElement> ::= '&'<BlockName> | '&'<Label>\r
@@ -366,7 +552,8 @@ InitializeRequestElement (
   IN OUT FORM_BROWSER_FORM        *Form\r
   )\r
 {\r
-  FORMSET_STORAGE  *Storage;\r
+  BROWSER_STORAGE  *Storage;\r
+  FORMSET_STORAGE  *FormsetStorage;\r
   UINTN            StrLen;\r
   UINTN            StringSize;\r
   CHAR16           *NewStr;\r
@@ -413,27 +600,43 @@ InitializeRequestElement (
     return EFI_SUCCESS;\r
   }\r
 \r
+  //\r
+  // Find Formset Storage for this Question\r
+  //\r
+  FormsetStorage = NULL;\r
+  Link = GetFirstNode (&FormSet->StorageListHead);\r
+  while (!IsNull (&FormSet->StorageListHead, Link)) {\r
+    FormsetStorage = FORMSET_STORAGE_FROM_LINK (Link);\r
+\r
+    if (FormsetStorage->VarStoreId == Question->VarStoreId) {\r
+      break;\r
+    }\r
+\r
+    Link = GetNextNode (&FormSet->StorageListHead, Link);\r
+  }\r
+  ASSERT (FormsetStorage != NULL);\r
+\r
   //\r
   // Append <RequestElement> to <ConfigRequest>\r
   //\r
-  if (StrLen > Storage->SpareStrLen) {\r
+  if (StrLen > FormsetStorage->SpareStrLen) {\r
     //\r
     // Old String buffer is not sufficient for RequestElement, allocate a new one\r
     //\r
-    StringSize = (Storage->ConfigRequest != NULL) ? StrSize (Storage->ConfigRequest) : sizeof (CHAR16);\r
+    StringSize = (FormsetStorage->ConfigRequest != NULL) ? StrSize (FormsetStorage->ConfigRequest) : sizeof (CHAR16);\r
     NewStr = AllocateZeroPool (StringSize + CONFIG_REQUEST_STRING_INCREMENTAL * sizeof (CHAR16));\r
     ASSERT (NewStr != NULL);\r
-    if (Storage->ConfigRequest != NULL) {\r
-      CopyMem (NewStr, Storage->ConfigRequest, StringSize);\r
-      FreePool (Storage->ConfigRequest);\r
+    if (FormsetStorage->ConfigRequest != NULL) {\r
+      CopyMem (NewStr, FormsetStorage->ConfigRequest, StringSize);\r
+      FreePool (FormsetStorage->ConfigRequest);\r
     }\r
-    Storage->ConfigRequest = NewStr;\r
-    Storage->SpareStrLen   = CONFIG_REQUEST_STRING_INCREMENTAL;\r
+    FormsetStorage->ConfigRequest = NewStr;\r
+    FormsetStorage->SpareStrLen   = CONFIG_REQUEST_STRING_INCREMENTAL;\r
   }\r
 \r
-  StrCat (Storage->ConfigRequest, RequestElement);\r
-  Storage->ElementCount++;\r
-  Storage->SpareStrLen -= StrLen;\r
+  StrCat (FormsetStorage->ConfigRequest, RequestElement);\r
+  FormsetStorage->ElementCount++;\r
+  FormsetStorage->SpareStrLen -= StrLen;\r
 \r
   //\r
   // Update the Config Request info saved in the form.\r
@@ -444,7 +647,7 @@ InitializeRequestElement (
   while (!IsNull (&Form->ConfigRequestHead, Link)) {\r
     ConfigInfo = FORM_BROWSER_CONFIG_REQUEST_FROM_LINK (Link);\r
 \r
-    if (ConfigInfo != NULL && ConfigInfo->Storage->VarStoreId == Storage->VarStoreId) {\r
+    if (ConfigInfo != NULL && ConfigInfo->Storage == Storage) {\r
       Find = TRUE;\r
       break;\r
     }\r
@@ -532,7 +735,6 @@ DestroyExpression (
   FreePool (Expression);\r
 }\r
 \r
-\r
 /**\r
   Free resources of a storage.\r
 \r
@@ -544,43 +746,10 @@ DestroyStorage (
   IN FORMSET_STORAGE   *Storage\r
   )\r
 {\r
-  LIST_ENTRY         *Link;\r
-  NAME_VALUE_NODE    *NameValueNode;\r
-\r
   if (Storage == NULL) {\r
     return;\r
   }\r
 \r
-  if (Storage->Name != NULL) {\r
-    FreePool (Storage->Name);\r
-  }\r
-  if (Storage->Buffer != NULL) {\r
-    FreePool (Storage->Buffer);\r
-  }\r
-  if (Storage->EditBuffer != NULL) {\r
-    FreePool (Storage->EditBuffer);\r
-  }\r
-\r
-  while (!IsListEmpty (&Storage->NameValueListHead)) {\r
-    Link = GetFirstNode (&Storage->NameValueListHead);\r
-    NameValueNode = NAME_VALUE_NODE_FROM_LINK (Link);\r
-    RemoveEntryList (&NameValueNode->Link);\r
-\r
-    if (NameValueNode->Name != NULL) {\r
-      FreePool (NameValueNode->Name);\r
-    }\r
-    if (NameValueNode->Value != NULL) {\r
-      FreePool (NameValueNode->Value);\r
-    }\r
-    if (NameValueNode->EditValue != NULL) {\r
-      FreePool (NameValueNode->EditValue);\r
-    }\r
-    FreePool (NameValueNode);\r
-  }\r
-\r
-  if (Storage->ConfigHdr != NULL) {\r
-    FreePool (Storage->ConfigHdr);\r
-  }\r
   if (Storage->ConfigRequest != NULL) {\r
     FreePool (Storage->ConfigRequest);\r
   }\r
@@ -654,6 +823,17 @@ DestroyStatement (
     DestroyExpression (Expression);\r
   }\r
 \r
+  //\r
+  // Free WarningIf List\r
+  //\r
+  while (!IsListEmpty (&Statement->WarningListHead)) {\r
+    Link = GetFirstNode (&Statement->WarningListHead);\r
+    Expression = FORM_EXPRESSION_FROM_LINK (Link);\r
+    RemoveEntryList (&Expression->Link);\r
+\r
+    DestroyExpression (Expression);\r
+  }\r
+\r
   if (Statement->Expression != NULL) {\r
     FreePool (Statement->Expression);\r
   }\r
@@ -856,6 +1036,39 @@ IsExpressionOpCode (
   }\r
 }\r
 \r
+/**\r
+  Tell whether this Operand is an Statement OpCode.\r
+\r
+  @param  Operand                Operand of an IFR OpCode.\r
+\r
+  @retval TRUE                   This is an Statement OpCode.\r
+  @retval FALSE                  Not an Statement OpCode.\r
+\r
+**/\r
+BOOLEAN\r
+IsStatementOpCode (\r
+  IN UINT8              Operand\r
+  )\r
+{\r
+  if ((Operand == EFI_IFR_SUBTITLE_OP) ||\r
+      (Operand == EFI_IFR_TEXT_OP) ||\r
+      (Operand == EFI_IFR_RESET_BUTTON_OP) ||\r
+      (Operand == EFI_IFR_REF_OP) ||\r
+      (Operand == EFI_IFR_ACTION_OP) ||\r
+      (Operand == EFI_IFR_NUMERIC_OP) ||\r
+      (Operand == EFI_IFR_ORDERED_LIST_OP) ||\r
+      (Operand == EFI_IFR_CHECKBOX_OP) ||\r
+      (Operand == EFI_IFR_STRING_OP) ||\r
+      (Operand == EFI_IFR_PASSWORD_OP) ||\r
+      (Operand == EFI_IFR_DATE_OP) ||\r
+      (Operand == EFI_IFR_TIME_OP) ||\r
+      (Operand == EFI_IFR_GUID_OP) ||\r
+      (Operand == EFI_IFR_ONE_OF_OP)) {\r
+    return TRUE;\r
+  } else {\r
+    return FALSE;\r
+  }\r
+}\r
 \r
 /**\r
   Calculate number of Statemens(Questions) and Expression OpCodes.\r
@@ -915,9 +1128,9 @@ ParseOpCodes (
   )\r
 {\r
   EFI_STATUS              Status;\r
-  UINT16                  Index;\r
   FORM_BROWSER_FORM       *CurrentForm;\r
   FORM_BROWSER_STATEMENT  *CurrentStatement;\r
+  FORM_BROWSER_STATEMENT  *ParentStatement;\r
   EXPRESSION_OPCODE       *ExpressionOpCode;\r
   FORM_EXPRESSION         *CurrentExpression;\r
   UINT8                   Operand;\r
@@ -931,7 +1144,6 @@ ParseOpCodes (
   QUESTION_DEFAULT        *CurrentDefault;\r
   QUESTION_OPTION         *CurrentOption;\r
   UINT8                   Width;\r
-  CHAR8                   *AsciiString;\r
   UINT16                  NumberOfStatement;\r
   UINT16                  NumberOfExpression;\r
   EFI_IMAGE_ID            *ImageId;\r
@@ -951,7 +1163,6 @@ ParseOpCodes (
   BOOLEAN                 InScopeDisable;\r
   INTN                    ConditionalExprCount;\r
 \r
-  mInScopeSubtitle         = FALSE;\r
   SuppressForQuestion      = FALSE;\r
   SuppressForOption        = FALSE;\r
   InScopeDisable           = FALSE;\r
@@ -977,6 +1188,7 @@ ParseOpCodes (
   CountOpCodes (FormSet, &NumberOfStatement, &NumberOfExpression);\r
 \r
   mStatementIndex = 0;\r
+  mUsedQuestionId = 1;\r
   FormSet->StatementBuffer = AllocateZeroPool (NumberOfStatement * sizeof (FORM_BROWSER_STATEMENT));\r
   if (FormSet->StatementBuffer == NULL) {\r
     return EFI_OUT_OF_RESOURCES;\r
@@ -988,6 +1200,7 @@ ParseOpCodes (
     return EFI_OUT_OF_RESOURCES;\r
   }\r
 \r
+  InitializeListHead (&FormSet->StatementListOSF);\r
   InitializeListHead (&FormSet->StorageListHead);\r
   InitializeListHead (&FormSet->DefaultStoreListHead);\r
   InitializeListHead (&FormSet->FormListHead);\r
@@ -997,6 +1210,7 @@ ParseOpCodes (
 \r
   CurrentForm = NULL;\r
   CurrentStatement = NULL;\r
+  ParentStatement  = NULL;\r
 \r
   ResetScopeStack ();\r
 \r
@@ -1087,8 +1301,8 @@ ParseOpCodes (
         break;\r
 \r
       case EFI_IFR_THIS_OP:\r
-        ASSERT (CurrentStatement != NULL);\r
-        ExpressionOpCode->QuestionId = CurrentStatement->QuestionId;\r
+        ASSERT (ParentStatement != NULL);\r
+        ExpressionOpCode->QuestionId = ParentStatement->QuestionId;\r
         break;\r
 \r
       case EFI_IFR_SECURITY_OP:\r
@@ -1104,7 +1318,7 @@ ParseOpCodes (
             while (!IsNull (&FormSet->StorageListHead, Link)) {\r
               VarStorage = FORMSET_STORAGE_FROM_LINK (Link);\r
               if (VarStorage->VarStoreId == ((EFI_IFR_GET *) OpCodeData)->VarStoreId) {\r
-                ExpressionOpCode->VarStorage = VarStorage;\r
+                ExpressionOpCode->VarStorage = VarStorage->BrowserStorage;\r
                 break;\r
               }\r
               Link = GetNextNode (&FormSet->StorageListHead, Link);\r
@@ -1346,7 +1560,6 @@ ParseOpCodes (
       InitializeListHead (&CurrentForm->ConfigRequestHead);\r
 \r
       CurrentForm->FormType = STANDARD_MAP_FORM_TYPE;\r
-      CurrentForm->NvUpdateRequired = FALSE;\r
       CopyMem (&CurrentForm->FormId,    &((EFI_IFR_FORM *) OpCodeData)->FormId,    sizeof (UINT16));\r
       CopyMem (&CurrentForm->FormTitle, &((EFI_IFR_FORM *) OpCodeData)->FormTitle, sizeof (EFI_STRING_ID));\r
 \r
@@ -1383,7 +1596,6 @@ ParseOpCodes (
       CurrentForm = AllocateZeroPool (sizeof (FORM_BROWSER_FORM));\r
       ASSERT (CurrentForm != NULL);\r
       CurrentForm->Signature = FORM_BROWSER_FORM_SIGNATURE;\r
-      CurrentForm->NvUpdateRequired = FALSE;\r
       InitializeListHead (&CurrentForm->ExpressionListHead);\r
       InitializeListHead (&CurrentForm->StatementListHead);\r
       InitializeListHead (&CurrentForm->ConfigRequestHead);\r
@@ -1448,76 +1660,34 @@ ParseOpCodes (
       //\r
       // Create a buffer Storage for this FormSet\r
       //\r
-      Storage = CreateStorage (FormSet);\r
-      Storage->Type = EFI_HII_VARSTORE_BUFFER;\r
-\r
+      Storage = CreateStorage (FormSet, EFI_HII_VARSTORE_BUFFER, OpCodeData);\r
       CopyMem (&Storage->VarStoreId, &((EFI_IFR_VARSTORE *) OpCodeData)->VarStoreId, sizeof (EFI_VARSTORE_ID));\r
-      CopyMem (&Storage->Guid,       &((EFI_IFR_VARSTORE *) OpCodeData)->Guid,       sizeof (EFI_GUID));\r
-      CopyMem (&Storage->Size,       &((EFI_IFR_VARSTORE *) OpCodeData)->Size,       sizeof (UINT16));\r
-\r
-      Storage->Buffer = AllocateZeroPool (Storage->Size);\r
-      Storage->EditBuffer = AllocateZeroPool (Storage->Size);\r
-\r
-      AsciiString = (CHAR8 *) ((EFI_IFR_VARSTORE *) OpCodeData)->Name;\r
-      Storage->Name = AllocateZeroPool (AsciiStrSize (AsciiString) * 2);\r
-      ASSERT (Storage->Name != NULL);\r
-      for (Index = 0; AsciiString[Index] != 0; Index++) {\r
-        Storage->Name[Index] = (CHAR16) AsciiString[Index];\r
-      }\r
-\r
-      //\r
-      // Initialize <ConfigHdr>\r
-      //\r
-      InitializeConfigHdr (FormSet, Storage);\r
       break;\r
 \r
     case EFI_IFR_VARSTORE_NAME_VALUE_OP:\r
       //\r
       // Create a name/value Storage for this FormSet\r
       //\r
-      Storage = CreateStorage (FormSet);\r
-      Storage->Type = EFI_HII_VARSTORE_NAME_VALUE;\r
-\r
+      Storage = CreateStorage (FormSet, EFI_HII_VARSTORE_NAME_VALUE, OpCodeData);\r
       CopyMem (&Storage->VarStoreId, &((EFI_IFR_VARSTORE_NAME_VALUE *) OpCodeData)->VarStoreId, sizeof (EFI_VARSTORE_ID));\r
-      CopyMem (&Storage->Guid,       &((EFI_IFR_VARSTORE_NAME_VALUE *) OpCodeData)->Guid,       sizeof (EFI_GUID));\r
-\r
-      //\r
-      // Initialize <ConfigHdr>\r
-      //\r
-      InitializeConfigHdr (FormSet, Storage);\r
       break;\r
 \r
     case EFI_IFR_VARSTORE_EFI_OP:\r
       //\r
       // Create a EFI variable Storage for this FormSet\r
       //\r
-      Storage = CreateStorage (FormSet);\r
-\r
-      CopyMem (&Storage->VarStoreId, &((EFI_IFR_VARSTORE_EFI *) OpCodeData)->VarStoreId, sizeof (EFI_VARSTORE_ID));\r
-      CopyMem (&Storage->Guid,       &((EFI_IFR_VARSTORE_EFI *) OpCodeData)->Guid,       sizeof (EFI_GUID));\r
-      CopyMem (&Storage->Attributes, &((EFI_IFR_VARSTORE_EFI *) OpCodeData)->Attributes, sizeof (UINT32));\r
-      CopyMem (&Storage->Size,       &((EFI_IFR_VARSTORE_EFI *) OpCodeData)->Size,       sizeof (UINT16));\r
-\r
       if (OpCodeLength < sizeof (EFI_IFR_VARSTORE_EFI)) {\r
-        Storage->Type = EFI_HII_VARSTORE_EFI_VARIABLE;\r
-        break;\r
-      } \r
-\r
-      Storage->Type = EFI_HII_VARSTORE_EFI_VARIABLE_BUFFER;\r
-      Storage->Buffer = AllocateZeroPool (Storage->Size);\r
-      Storage->EditBuffer = AllocateZeroPool (Storage->Size);\r
-\r
-      AsciiString = (CHAR8 *) ((EFI_IFR_VARSTORE_EFI *) OpCodeData)->Name;\r
-      Storage->Name = AllocateZeroPool (AsciiStrSize (AsciiString) * 2);\r
-      ASSERT (Storage->Name != NULL);\r
-      for (Index = 0; AsciiString[Index] != 0; Index++) {\r
-        Storage->Name[Index] = (CHAR16) AsciiString[Index];\r
+        //\r
+        // Create efi varstore with format follow UEFI spec before 2.3.1.\r
+        //\r
+        Storage = CreateStorage (FormSet, EFI_HII_VARSTORE_EFI_VARIABLE, OpCodeData);\r
+      } else {\r
+        //\r
+        // Create efi varstore with format follow UEFI spec 2.3.1 and later.\r
+        //\r
+        Storage = CreateStorage (FormSet, EFI_HII_VARSTORE_EFI_VARIABLE_BUFFER, OpCodeData);\r
       }\r
-\r
-      //\r
-      // Initialize <ConfigHdr>\r
-      //\r
-      InitializeConfigHdr (FormSet, Storage);\r
+      CopyMem (&Storage->VarStoreId, &((EFI_IFR_VARSTORE_EFI *) OpCodeData)->VarStoreId, sizeof (EFI_VARSTORE_ID));\r
       break;\r
 \r
     //\r
@@ -1545,22 +1715,20 @@ ParseOpCodes (
       ASSERT (CurrentStatement != NULL);\r
 \r
       CurrentStatement->Flags = ((EFI_IFR_SUBTITLE *) OpCodeData)->Flags;\r
-\r
-      if (Scope != 0) {\r
-        mInScopeSubtitle = TRUE;\r
-      }\r
+      CurrentStatement->FakeQuestionId = mUsedQuestionId++;\r
       break;\r
 \r
     case EFI_IFR_TEXT_OP:\r
       CurrentStatement = CreateStatement (OpCodeData, FormSet, CurrentForm);\r
       ASSERT (CurrentStatement != NULL);\r
-\r
+      CurrentStatement->FakeQuestionId = mUsedQuestionId++;\r
       CopyMem (&CurrentStatement->TextTwo, &((EFI_IFR_TEXT *) OpCodeData)->TextTwo, sizeof (EFI_STRING_ID));\r
       break;\r
 \r
     case EFI_IFR_RESET_BUTTON_OP:\r
       CurrentStatement = CreateStatement (OpCodeData, FormSet, CurrentForm);\r
       ASSERT (CurrentStatement != NULL);\r
+      CurrentStatement->FakeQuestionId = mUsedQuestionId++;\r
       CopyMem (&CurrentStatement->DefaultId, &((EFI_IFR_RESET_BUTTON *) OpCodeData)->DefaultId, sizeof (EFI_DEFAULT_ID));\r
       break;\r
 \r
@@ -1787,7 +1955,7 @@ ParseOpCodes (
       //\r
       // Insert to Default Value list of current Question\r
       //\r
-      InsertTailList (&CurrentStatement->DefaultListHead, &CurrentDefault->Link);\r
+      InsertTailList (&ParentStatement->DefaultListHead, &CurrentDefault->Link);\r
 \r
       if (Scope != 0) {\r
         InScopeDefault = TRUE;\r
@@ -1805,11 +1973,12 @@ ParseOpCodes (
       CurrentOption = AllocateZeroPool (sizeof (QUESTION_OPTION));\r
       ASSERT (CurrentOption != NULL);\r
       CurrentOption->Signature = QUESTION_OPTION_SIGNATURE;\r
+      CurrentOption->OpCode    = (EFI_IFR_ONE_OF_OPTION *) OpCodeData;\r
 \r
       CurrentOption->Flags = ((EFI_IFR_ONE_OF_OPTION *) OpCodeData)->Flags;\r
       CurrentOption->Value.Type = ((EFI_IFR_ONE_OF_OPTION *) OpCodeData)->Type;\r
       CopyMem (&CurrentOption->Text, &((EFI_IFR_ONE_OF_OPTION *) OpCodeData)->Option, sizeof (EFI_STRING_ID));\r
-      CopyMem (&CurrentOption->Value.Value, &((EFI_IFR_ONE_OF_OPTION *) OpCodeData)->Value, sizeof (EFI_IFR_TYPE_VALUE));\r
+      CopyMem (&CurrentOption->Value.Value, &((EFI_IFR_ONE_OF_OPTION *) OpCodeData)->Value, OpCodeLength - OFFSET_OF (EFI_IFR_ONE_OF_OPTION, Value));\r
       ExtendValueToU64 (&CurrentOption->Value);\r
 \r
       ConditionalExprCount = GetConditionalExpressionCount(ExpressOption);\r
@@ -1825,16 +1994,15 @@ ParseOpCodes (
         CopyMem (CurrentOption->SuppressExpression->Expression, GetConditionalExpressionList(ExpressOption), (UINTN) (sizeof (FORM_EXPRESSION *) * ConditionalExprCount));\r
       }\r
 \r
+      ASSERT (ParentStatement != NULL);\r
       //\r
       // Insert to Option list of current Question\r
       //\r
-      InsertTailList (&CurrentStatement->OptionListHead, &CurrentOption->Link);\r
-\r
+      InsertTailList (&ParentStatement->OptionListHead, &CurrentOption->Link);\r
       //\r
       // Now we know the Storage width of nested Ordered List\r
       //\r
-      ASSERT (CurrentStatement != NULL);\r
-      if ((CurrentStatement->Operand == EFI_IFR_ORDERED_LIST_OP) && (CurrentStatement->BufferValue == NULL)) {\r
+      if ((ParentStatement->Operand == EFI_IFR_ORDERED_LIST_OP) && (ParentStatement->BufferValue == NULL)) {\r
         Width = 1;\r
         switch (CurrentOption->Value.Type) {\r
         case EFI_IFR_TYPE_NUM_SIZE_8:\r
@@ -1860,15 +2028,15 @@ ParseOpCodes (
           break;\r
         }\r
 \r
-        CurrentStatement->StorageWidth = (UINT16) (CurrentStatement->MaxContainers * Width);\r
-        CurrentStatement->BufferValue = AllocateZeroPool (CurrentStatement->StorageWidth);\r
-        CurrentStatement->ValueType = CurrentOption->Value.Type;\r
-        if (CurrentStatement->HiiValue.Type == EFI_IFR_TYPE_BUFFER) {\r
-          CurrentStatement->HiiValue.Buffer = CurrentStatement->BufferValue;\r
-          CurrentStatement->HiiValue.BufferLen = CurrentStatement->StorageWidth;\r
+        ParentStatement->StorageWidth = (UINT16) (ParentStatement->MaxContainers * Width);\r
+        ParentStatement->BufferValue = AllocateZeroPool (ParentStatement->StorageWidth);\r
+        ParentStatement->ValueType = CurrentOption->Value.Type;\r
+        if (ParentStatement->HiiValue.Type == EFI_IFR_TYPE_BUFFER) {\r
+          ParentStatement->HiiValue.Buffer = ParentStatement->BufferValue;\r
+          ParentStatement->HiiValue.BufferLen = ParentStatement->StorageWidth;\r
         }\r
 \r
-        InitializeRequestElement (FormSet, CurrentStatement, CurrentForm);\r
+        InitializeRequestElement (FormSet, ParentStatement, CurrentForm);\r
       }\r
       break;\r
 \r
@@ -1885,10 +2053,10 @@ ParseOpCodes (
 \r
       if (Operand == EFI_IFR_NO_SUBMIT_IF_OP) {\r
         CurrentExpression->Type = EFI_HII_EXPRESSION_NO_SUBMIT_IF;\r
-        InsertTailList (&CurrentStatement->NoSubmitListHead, &CurrentExpression->Link);\r
+        InsertTailList (&ParentStatement->NoSubmitListHead, &CurrentExpression->Link);\r
       } else {\r
         CurrentExpression->Type = EFI_HII_EXPRESSION_INCONSISTENT_IF;\r
-        InsertTailList (&CurrentStatement->InconsistentListHead, &CurrentExpression->Link);\r
+        InsertTailList (&ParentStatement->InconsistentListHead, &CurrentExpression->Link);\r
       }\r
 \r
       //\r
@@ -1900,6 +2068,25 @@ ParseOpCodes (
       }\r
       break;\r
 \r
+    case EFI_IFR_WARNING_IF_OP:\r
+      //\r
+      // Create an Expression node\r
+      //\r
+      CurrentExpression = CreateExpression (CurrentForm);\r
+      CopyMem (&CurrentExpression->Error, &((EFI_IFR_WARNING_IF *) OpCodeData)->Warning, sizeof (EFI_STRING_ID));\r
+      CurrentExpression->TimeOut = ((EFI_IFR_WARNING_IF *) OpCodeData)->TimeOut;\r
+      CurrentExpression->Type    = EFI_HII_EXPRESSION_WARNING_IF;\r
+      InsertTailList (&ParentStatement->WarningListHead, &CurrentExpression->Link);\r
+\r
+      //\r
+      // Take a look at next OpCode to see whether current expression consists\r
+      // of single OpCode\r
+      //\r
+      if (((EFI_IFR_OP_HEADER *) (OpCodeData + OpCodeLength))->Scope == 0) {\r
+        SingleOpCodeExpression = TRUE;\r
+      }\r
+      break;\r
+\r
     case EFI_IFR_SUPPRESS_IF_OP:\r
       //\r
       // Question and Option will appear in scope of this OpCode\r
@@ -2000,8 +2187,8 @@ ParseOpCodes (
         // If it is NULL, 1) ParseOpCodes functions may parse the IFR wrongly. Or 2) the IFR\r
         // file is wrongly generated by tools such as VFR Compiler. There may be a bug in VFR Compiler.\r
         //\r
-        ASSERT (CurrentStatement != NULL);\r
-        CurrentStatement->ValueExpression = CurrentExpression;\r
+        ASSERT (ParentStatement != NULL);\r
+        ParentStatement->ValueExpression = CurrentExpression;\r
       }\r
 \r
       //\r
@@ -2039,8 +2226,8 @@ ParseOpCodes (
       // If it is NULL, 1) ParseOpCodes functions may parse the IFR wrongly. Or 2) the IFR\r
       // file is wrongly generated by tools such as VFR Compiler. There may be a bug in VFR Compiler.\r
       //\r
-      ASSERT (CurrentStatement != NULL);\r
-      CurrentStatement->ReadExpression = CurrentExpression;\r
+      ASSERT (ParentStatement != NULL);\r
+      ParentStatement->ReadExpression = CurrentExpression;\r
 \r
       //\r
       // Take a look at next OpCode to see whether current expression consists\r
@@ -2061,8 +2248,8 @@ ParseOpCodes (
       // If it is NULL, 1) ParseOpCodes functions may parse the IFR wrongly. Or 2) the IFR\r
       // file is wrongly generated by tools such as VFR Compiler. There may be a bug in VFR Compiler.\r
       //\r
-      ASSERT (CurrentStatement != NULL);\r
-      CurrentStatement->WriteExpression = CurrentExpression;\r
+      ASSERT (ParentStatement != NULL);\r
+      ParentStatement->WriteExpression = CurrentExpression;\r
 \r
       //\r
       // Take a look at next OpCode to see whether current expression consists\r
@@ -2104,8 +2291,8 @@ ParseOpCodes (
         // If it is NULL, 1) ParseOpCodes functions may parse the IFR wrongly. Or 2) the IFR\r
         // file is wrongly generated by tools such as VFR Compiler.\r
         //\r
-        ASSERT (CurrentStatement != NULL);\r
-        ImageId = &CurrentStatement->ImageId;\r
+        ASSERT (ParentStatement != NULL);\r
+        ImageId = &ParentStatement->ImageId;\r
         break;\r
       }\r
 \r
@@ -2117,16 +2304,16 @@ ParseOpCodes (
     // Refresh\r
     //\r
     case EFI_IFR_REFRESH_OP:\r
-      ASSERT (CurrentStatement != NULL);\r
-      CurrentStatement->RefreshInterval = ((EFI_IFR_REFRESH *) OpCodeData)->RefreshInterval;\r
+      ASSERT (ParentStatement != NULL);\r
+      ParentStatement->RefreshInterval = ((EFI_IFR_REFRESH *) OpCodeData)->RefreshInterval;\r
       break;\r
 \r
     //\r
     // Refresh guid.\r
     //\r
     case EFI_IFR_REFRESH_ID_OP:\r
-      ASSERT (CurrentStatement != NULL);\r
-      CopyMem (&CurrentStatement->RefreshGuid, &((EFI_IFR_REFRESH_ID *) OpCodeData)->RefreshEventGroupId, sizeof (EFI_GUID));\r
+      ASSERT (ParentStatement != NULL);\r
+      CopyMem (&ParentStatement->RefreshGuid, &((EFI_IFR_REFRESH_ID *) OpCodeData)->RefreshEventGroupId, sizeof (EFI_GUID));\r
       break;\r
 \r
     //\r
@@ -2154,53 +2341,16 @@ ParseOpCodes (
         break;\r
 \r
       default:\r
-        ASSERT (CurrentStatement != NULL);\r
-        CurrentStatement->Locked = TRUE;\r
+        ASSERT (ParentStatement != NULL);\r
+        ParentStatement->Locked = TRUE;\r
       }      \r
       break;\r
 \r
     //\r
     // Vendor specific\r
     //\r
-    case EFI_IFR_GUID_OP:\r
-      if (CompareGuid (&gEfiIfrTianoGuid, (EFI_GUID *)(OpCodeData + sizeof (EFI_IFR_OP_HEADER)))) {\r
-        //\r
-        // Tiano specific GUIDed opcodes\r
-        //\r
-        switch (((EFI_IFR_GUID_LABEL *) OpCodeData)->ExtendOpCode) {\r
-        case EFI_IFR_EXTEND_OP_LABEL:\r
-          //\r
-          // just ignore label\r
-          //\r
-          break;\r
-\r
-        case EFI_IFR_EXTEND_OP_BANNER:\r
-          //\r
-          // By SubClass to get Banner Data from Front Page\r
-          //\r
-          if (FormSet->SubClass == EFI_FRONT_PAGE_SUBCLASS) {\r
-            CopyMem (\r
-              &gBannerData->Banner[((EFI_IFR_GUID_BANNER *) OpCodeData)->LineNumber][\r
-              ((EFI_IFR_GUID_BANNER *) OpCodeData)->Alignment],\r
-              &((EFI_IFR_GUID_BANNER *) OpCodeData)->Title,\r
-              sizeof (EFI_STRING_ID)\r
-              );\r
-          }\r
-          break;\r
-\r
-        case EFI_IFR_EXTEND_OP_CLASS:\r
-          CopyMem (&FormSet->Class, &((EFI_IFR_GUID_CLASS *) OpCodeData)->Class, sizeof (UINT16));\r
-          break;\r
-\r
-        case EFI_IFR_EXTEND_OP_SUBCLASS:\r
-          CopyMem (&FormSet->SubClass, &((EFI_IFR_GUID_SUBCLASS *) OpCodeData)->SubClass, sizeof (UINT16));\r
-          break;\r
-\r
-        default:\r
-          break;\r
-        }\r
-      }\r
-\r
+    case EFI_IFR_GUID_OP:     \r
+      CurrentStatement = CreateStatement (OpCodeData, FormSet, CurrentForm);\r
       break;\r
 \r
     //\r
@@ -2212,6 +2362,13 @@ ParseOpCodes (
         ResetScopeStack ();\r
         return Status;\r
       }\r
+      \r
+      //\r
+      // Parent statement end tag found, update ParentStatement info.\r
+      //\r
+      if (IsStatementOpCode(ScopeOpCode) && (ParentStatement != NULL) && (ParentStatement->Operand == ScopeOpCode)) {\r
+        ParentStatement  = ParentStatement->ParentStatement;\r
+      }\r
 \r
       switch (ScopeOpCode) {\r
       case EFI_IFR_FORM_SET_OP:\r
@@ -2238,12 +2395,9 @@ ParseOpCodes (
         CurrentOption = NULL;\r
         break;\r
 \r
-      case EFI_IFR_SUBTITLE_OP:\r
-        mInScopeSubtitle = FALSE;\r
-        break;\r
-\r
       case EFI_IFR_NO_SUBMIT_IF_OP:\r
       case EFI_IFR_INCONSISTENT_IF_OP:\r
+      case EFI_IFR_WARNING_IF_OP:\r
         //\r
         // Ignore end of EFI_IFR_NO_SUBMIT_IF and EFI_IFR_INCONSISTENT_IF\r
         //\r
@@ -2332,6 +2486,17 @@ ParseOpCodes (
     default:\r
       break;\r
     }\r
+\r
+    if (IsStatementOpCode(Operand)) {\r
+      CurrentStatement->ParentStatement = ParentStatement;\r
+      if (Scope != 0) {\r
+        //\r
+        // Scope != 0, other statements or options may nest in this statement.\r
+        // Update the ParentStatement info.\r
+        //\r
+        ParentStatement = CurrentStatement;\r
+      }\r
+    }\r
   }\r
 \r
   return EFI_SUCCESS;\r