commit bab9f949bf1a72f33f89331bbffa9362de254945
author Qiu Shumin <shumin.qiu@intel.com> Thu Nov 20 01:31:36 2014 +0000
committer shenshushi <shenshushi@Edk2> Thu Nov 20 01:31:36 2014 +0000
tree 1472e6bce47568799d8560dac33f05ddbceec3bf
parent 771ee5017be405df3a05ae8dce67552cab3b18b1

ShellPkg: Code refine to avoid potential dereference of pointer.

Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Qiu Shumin <shumin.qiu@intel.com>
Reviewed-by: Jaben Carsey <Jaben.carsey@intel.com>


git-svn-id: https://svn.code.sf.net/p/edk2/code/trunk/edk2@16410 6f19259b-4bc3-4df7-8a09-765794883524

ShellPkg/Library/UefiShellLib/UefiShellLib.c

diff --git a/ShellPkg/Library/UefiShellLib/UefiShellLib.c b/ShellPkg/Library/UefiShellLib/UefiShellLib.c
index 665e733..6cef8b7 100644
--- a/ShellPkg/Library/UefiShellLib/UefiShellLib.c
+++ b/ShellPkg/Library/UefiShellLib/UefiShellLib.c
@@ -2078,7 +2078,7 @@
           ASSERT(GetItemValue == 0);

           break;

       }

-    } else if (GetItemValue != 0 && !InternalIsFlag(Argv[LoopCounter], AlwaysAllowNumbers, (CONST BOOLEAN)(CurrentItemPackage->Type == TypeTimeValue))) {

+    } else if (GetItemValue != 0 && CurrentItemPackage != NULL && !InternalIsFlag(Argv[LoopCounter], AlwaysAllowNumbers, (CONST BOOLEAN)(CurrentItemPackage->Type == TypeTimeValue))) {

       ASSERT(CurrentItemPackage != NULL);

       //

       // get the item VALUE for a previous flag