ShellPkg: Eliminate loss of high bits in return value from exit command on 64 bit platform.

submitted-by: jcarsey
reviewed-by:  gookboy15a
reviewed-by:  niruiyu

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

3 files changed, 26 insertions, 7 deletions

diff --git a/ShellPkg/Library/UefiShellCommandLib/UefiShellCommandLib.c b/ShellPkg/Library/UefiShellCommandLib/UefiShellCommandLib.c
index 2c1dcd3c13..971294ffd4 100644
--- a/ShellPkg/Library/UefiShellCommandLib/UefiShellCommandLib.c
+++ b/ShellPkg/Library/UefiShellCommandLib/UefiShellCommandLib.c
@@ -26,6 +26,7 @@ STATIC SCRIPT_FILE_LIST                   mScriptList;
 STATIC ALIAS_LIST                         mAliasList;

 STATIC BOOLEAN                            mEchoState;

 STATIC BOOLEAN                            mExitRequested;

+STATIC UINT64                             mExitCode;

 STATIC BOOLEAN                            mExitScript;

 STATIC CHAR16                             *mProfileList;

 STATIC UINTN                              mProfileListSize;

@@ -697,12 +698,14 @@ ShellCommandSetEchoState(
 /**

   Indicate that the current shell or script should exit.

 

-  @param[in] ScriptOnly   TRUE if only exiting a script, FALSE othrwise.

+  @param[in] ScriptOnly   TRUE if exiting a script; FALSE otherwise.

+  @param[in] ErrorCode    The 64 bit error code to return.

 **/

 VOID

 EFIAPI

 ShellCommandRegisterExit (

-  IN BOOLEAN ScriptOnly

+  IN BOOLEAN      ScriptOnly,

+  IN CONST UINT64 ErrorCode

   )

 {

   mExitRequested = (BOOLEAN)(!mExitRequested);

@@ -711,6 +714,7 @@ ShellCommandRegisterExit (
   } else {

     mExitScript    = FALSE;

   }

+  mExitCode = ErrorCode;

 }

 

 /**

@@ -729,6 +733,21 @@ ShellCommandGetExit (
 }

 

 /**

+  Retrieve the Exit code.

+

+  If ShellCommandGetExit returns FALSE than the return from this is undefined.

+

+  @return the value passed into RegisterExit.

+**/

+UINT64

+EFIAPI

+ShellCommandGetExitCode (

+  VOID

+  )

+{

+  return (mExitCode);

+}

+/**

   Retrieve the Exit script indicator.

 

   If ShellCommandGetExit returns FALSE than the return from this is undefined.

diff --git a/ShellPkg/Library/UefiShellLevel1CommandsLib/Exit.c b/ShellPkg/Library/UefiShellLevel1CommandsLib/Exit.c
index 5620d023b1..3a3c59fa03 100644
--- a/ShellPkg/Library/UefiShellLevel1CommandsLib/Exit.c
+++ b/ShellPkg/Library/UefiShellLevel1CommandsLib/Exit.c
@@ -77,16 +77,16 @@ ShellCommandRunExit (
         //

         // If we are in a batch file and /b then pass TRUE otherwise false...

         //

-        ShellCommandRegisterExit((BOOLEAN)(gEfiShellProtocol->BatchIsActive() && ShellCommandLineGetFlag(Package, L"/b")));

+        ShellCommandRegisterExit((BOOLEAN)(gEfiShellProtocol->BatchIsActive() && ShellCommandLineGetFlag(Package, L"/b")), RetVal);

 

-        ShellStatus = (SHELL_STATUS)(RetVal);

+        ShellStatus = SHELL_SUCCESS;

       }

     } else {

       // If we are in a batch file and /b then pass TRUE otherwise false...

       //

-      ShellCommandRegisterExit((BOOLEAN)(gEfiShellProtocol->BatchIsActive() && ShellCommandLineGetFlag(Package, L"/b")));

+      ShellCommandRegisterExit((BOOLEAN)(gEfiShellProtocol->BatchIsActive() && ShellCommandLineGetFlag(Package, L"/b")), 0);

 

-      ShellStatus = (SHELL_STATUS)0;

+      ShellStatus = SHELL_SUCCESS;

     }

 

     ShellCommandLineFreeVarList (Package);

diff --git a/ShellPkg/Library/UefiShellLevel3CommandsLib/Pause.c b/ShellPkg/Library/UefiShellLevel3CommandsLib/Pause.c
index d6ed793579..bedd591c0d 100644
--- a/ShellPkg/Library/UefiShellLevel3CommandsLib/Pause.c
+++ b/ShellPkg/Library/UefiShellLevel3CommandsLib/Pause.c
@@ -84,7 +84,7 @@ ShellCommandRunPause (
       }

 

       if (EFI_ERROR(Status) || Resp == NULL || *Resp == ShellPromptResponseQuit) {

-        ShellCommandRegisterExit(TRUE);

+        ShellCommandRegisterExit(TRUE, 0);

         ShellStatus = SHELL_ABORTED;

       }