From: xli24 <xli24@6f19259b-4bc3-4df7-8a09-765794883524>
Date: Thu, 3 Jun 2010 08:49:46 +0000 (+0000)
Subject: Fix the issue that ASSERT() at line 1391 incorrectly asserts when order list variable... 
X-Git-Tag: edk2-stable201903~15780
X-Git-Url: https://git.proxmox.com/?p=mirror_edk2.git;a=commitdiff_plain;h=c6498c1d459dbdedead7664fd3c6e498a337e4f1

Fix the issue that ASSERT() at line 1391 incorrectly asserts when order list variable is empty.

git-svn-id: https://edk2.svn.sourceforge.net/svnroot/edk2/trunk/edk2@10564 6f19259b-4bc3-4df7-8a09-765794883524
---

diff --git a/IntelFrameworkModulePkg/Universal/BdsDxe/BootMaint/BootOption.c b/IntelFrameworkModulePkg/Universal/BdsDxe/BootMaint/BootOption.c
index 1ca93fd886..2cfc3b817a 100644
--- a/IntelFrameworkModulePkg/Universal/BdsDxe/BootMaint/BootOption.c
+++ b/IntelFrameworkModulePkg/Universal/BdsDxe/BootMaint/BootOption.c
@@ -1380,6 +1380,7 @@ BOpt_GetOptionNumber (
   OrderListSize = 0;
   OrderList     = NULL;
   OptionNumber  = 0;
+  Index         = 0;
 
   UnicodeSPrint (StrTemp, sizeof (StrTemp), L"%sOrder", Type);
 
@@ -1388,12 +1389,13 @@ BOpt_GetOptionNumber (
                           &gEfiGlobalVariableGuid,
                           &OrderListSize
                           );
-  ASSERT (OrderList != NULL);
 
   for (OptionNumber = 0; ; OptionNumber++) {
-    for (Index = 0; Index < OrderListSize / sizeof (UINT16); Index++) {
-      if (OptionNumber == OrderList[Index]) {
-        break;
+    if (OrderList != NULL) {
+      for (Index = 0; Index < OrderListSize / sizeof (UINT16); Index++) {
+        if (OptionNumber == OrderList[Index]) {
+          break;
+        }
       }
     }