[Cherry-Pick] [RB&FF] MdeModulePkg/HiiDatabaseDxe: Remove assert for …

…VarStoreId = 0 (#1185) ## Description It is legal for the VarStoreId of a question to be 0 per the UEFI spec: "Specifies the identifier of a previously declared variable store to use when storing the question’s value. A value of zero indicates no associated variable store." Instead of hitting an assert just skip this question as there is no value to return. Signed-off-by: Jeff Brasen <[email protected]> Co-authored-by: Jeff Brasen <[email protected]>
microsoft · Oct 16, 2024 · 27e1a98 · 27e1a98
1 parent a45519a
commit 27e1a98
Showing 1 changed file with 3 additions and 2 deletions.
diff --git a/MdeModulePkg/Universal/HiiDatabaseDxe/ConfigKeywordHandler.c b/MdeModulePkg/Universal/HiiDatabaseDxe/ConfigKeywordHandler.c
@@ -2119,8 +2119,9 @@ ExtractConfigRequest (
       //
       // Header->VarStoreId == 0 means no storage for this question.
       //
-      ASSERT (Header->VarStoreId != 0);
-      DEBUG ((DEBUG_INFO, "Varstore Id: 0x%x\n", Header->VarStoreId));
+      if (Header->VarStoreId == 0) {
+        continue;
+      }
 
       Storage = FindStorageFromVarId (FormPackage, Header->VarStoreId);
       ASSERT (Storage != NULL);