commit 919278054f43c410268941d5970e533a7596bcab
author Dandan Bi <dandan.bi@intel.com> Mon Dec 05 19:18:13 2016 +0800
committer Star Zeng <star.zeng@intel.com> Tue Dec 06 18:41:43 2016 +0800
tree d93eb2f345e5620e9164b307d571509209b1aee3
parent d138a2e987c51d8225ffaba5b523db82f75269cd

MdeModulePkg/SetupBrowserDxe: Fix incorrect local variable used

https://bugzilla.tianocore.org/show_bug.cgi?id=280

Cc: Eric Dong <eric.dong@intel.com>
Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Dandan Bi <dandan.bi@intel.com>
Reviewed-by: Eric Dong <eric.dong@intel.com>

MdeModulePkg/Universal/SetupBrowserDxe/Setup.c

diff --git a/MdeModulePkg/Universal/SetupBrowserDxe/Setup.c b/MdeModulePkg/Universal/SetupBrowserDxe/Setup.c
index 451fd03..06e682a 100644
--- a/MdeModulePkg/Universal/SetupBrowserDxe/Setup.c
+++ b/MdeModulePkg/Universal/SetupBrowserDxe/Setup.c
@@ -3535,6 +3535,7 @@
 {

   EFI_STATUS              Status;

   LIST_ENTRY              *Link;

+  LIST_ENTRY              *FormLink;

   LIST_ENTRY              *StorageLink;

   FORMSET_STORAGE         *FormSetStorage;

   FORM_BROWSER_FORM       *Form;

@@ -3625,10 +3626,10 @@
         }

       }

 

-      Link = GetFirstNode (&LocalFormSet->FormListHead);

-      while (!IsNull (&LocalFormSet->FormListHead, Link)) {

-        Form = FORM_BROWSER_FORM_FROM_LINK (Link);

-        Link = GetNextNode (&LocalFormSet->FormListHead, Link);

+      FormLink = GetFirstNode (&LocalFormSet->FormListHead);

+      while (!IsNull (&LocalFormSet->FormListHead, FormLink)) {

+        Form = FORM_BROWSER_FORM_FROM_LINK (FormLink);

+        FormLink = GetNextNode (&LocalFormSet->FormListHead, FormLink);

         //

         // Call callback with Changed type to inform the driver.

         //