From mboxrd@z Thu Jan 1 00:00:00 1970 Authentication-Results: mx.groups.io; dkim=missing; spf=pass (domain: intel.com, ip: 192.55.52.93, mailfrom: zhichao.gao@intel.com) Received: from mga11.intel.com (mga11.intel.com [192.55.52.93]) by groups.io with SMTP; Wed, 25 Sep 2019 19:43:42 -0700 X-Amp-Result: SKIPPED(no attachment in message) X-Amp-File-Uploaded: False Received: from fmsmga007.fm.intel.com ([10.253.24.52]) by fmsmga102.fm.intel.com with ESMTP/TLS/DHE-RSA-AES256-GCM-SHA384; 25 Sep 2019 19:43:42 -0700 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.64,550,1559545200"; d="scan'208";a="189862376" Received: from fmsmsx107.amr.corp.intel.com ([10.18.124.205]) by fmsmga007.fm.intel.com with ESMTP; 25 Sep 2019 19:43:42 -0700 Received: from fmsmsx126.amr.corp.intel.com (10.18.125.43) by fmsmsx107.amr.corp.intel.com (10.18.124.205) with Microsoft SMTP Server (TLS) id 14.3.439.0; Wed, 25 Sep 2019 19:43:42 -0700 Received: from shsmsx151.ccr.corp.intel.com (10.239.6.50) by FMSMSX126.amr.corp.intel.com (10.18.125.43) with Microsoft SMTP Server (TLS) id 14.3.439.0; Wed, 25 Sep 2019 19:43:41 -0700 Received: from shsmsx101.ccr.corp.intel.com ([169.254.1.92]) by SHSMSX151.ccr.corp.intel.com ([169.254.3.53]) with mapi id 14.03.0439.000; Thu, 26 Sep 2019 10:43:38 +0800 From: "Gao, Zhichao" To: Laszlo Ersek , edk2-devel-groups-io CC: "Carsey, Jaben" , "Ni, Ray" Subject: Re: [PATCH 29/35] ShellPkg: stop using EFI_HANDLE in place of EFI_HII_HANDLE Thread-Topic: [PATCH 29/35] ShellPkg: stop using EFI_HANDLE in place of EFI_HII_HANDLE Thread-Index: AQHVbZE3CpAWiX/LVUeJ8vEEfqXMg6c9TPzA Date: Thu, 26 Sep 2019 02:43:38 +0000 Message-ID: <3CE959C139B4C44DBEA1810E3AA6F9000B83F0CF@SHSMSX101.ccr.corp.intel.com> References: <20190917194935.24322-1-lersek@redhat.com> <20190917194935.24322-30-lersek@redhat.com> In-Reply-To: <20190917194935.24322-30-lersek@redhat.com> Accept-Language: en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: x-originating-ip: [10.239.127.40] MIME-Version: 1.0 Return-Path: zhichao.gao@intel.com Content-Language: en-US Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: quoted-printable Reviewed-by: Zhichao Gao > -----Original Message----- > From: Laszlo Ersek [mailto:lersek@redhat.com] > Sent: Wednesday, September 18, 2019 3:49 AM > To: edk2-devel-groups-io > Cc: Carsey, Jaben ; Ni, Ray ; > Gao, Zhichao > Subject: [PATCH 29/35] ShellPkg: stop using EFI_HANDLE in place of > EFI_HII_HANDLE >=20 > The UefiShell*CommandsLib instances have constructor functions that do > something like: >=20 > gHiiHandle =3D HiiAddPackages (...); > ... > ShellCommandRegisterCommandName (..., gHiiHandle, ...); >=20 > and destructor functions that implement the following pattern: >=20 > HiiRemovePackages (gHiiHandle); >=20 > The -- semantic, not functional -- problem is that "gHiiHandle" is declar= ed > with type EFI_HANDLE, and not EFI_HII_HANDLE, in all of these library > instances, even though HiiAddPackages() correctly returns EFI_HII_HANDLE, > and HiiRemovePackages() takes EFI_HII_HANDLE. >=20 > Once we fix the type of "gHiiHandle", it causes sort of a butterfly effec= t, > because it is passed around widely. Track down and update all of those > locations. >=20 > The DynamicCommand lib instances use a similar pattern, so they are > affected too. >=20 > NOTE: in practice, this patch is a no-op, as both EFI_HII_HANDLE and > EFI_HANDLE are typedefs to (VOID*). However, we shouldn't use > EFI_HANDLE where semantically EFI_HII_HANDLE is passed around. >=20 > Cc: Jaben Carsey > Cc: Ray Ni > Cc: Zhichao Gao > Signed-off-by: Laszlo Ersek > --- >=20 > Notes: > tested with: > - level 1: stall, exit > - level 2: ls > - level 3: pause > - dynamic command: tftp > - handle parsing: drivers, devices, dh >=20 > ShellPkg/Include/Library/ShellCommandLib.h = | 2 +- > ShellPkg/Include/Library/ShellLib.h = | 4 ++-- > ShellPkg/DynamicCommand/DpDynamicCommand/Dp.h = | 4 > ++-- > ShellPkg/DynamicCommand/TftpDynamicCommand/Tftp.h = | 4 > ++-- > ShellPkg/Library/UefiShellCommandLib/UefiShellCommandLib.h = | 2 > +- >=20 > ShellPkg/Library/UefiShellDebug1CommandsLib/UefiShellDebug1Commands > Lib.h | 2 +- >=20 > ShellPkg/Library/UefiShellDriver1CommandsLib/UefiShellDriver1CommandsL > ib.h | 2 +- >=20 > ShellPkg/Library/UefiShellLevel1CommandsLib/UefiShellLevel1CommandsLib > .h | 2 +- >=20 > ShellPkg/Library/UefiShellLevel2CommandsLib/UefiShellLevel2CommandsLib > .h | 2 +- >=20 > ShellPkg/Library/UefiShellLevel3CommandsLib/UefiShellLevel3CommandsLib > .h | 2 +- >=20 > ShellPkg/Library/UefiShellNetwork1CommandsLib/UefiShellNetwork1Comm > andsLib.h | 2 +- > ShellPkg/Library/UefiShellNetwork2CommandsLib/UefiShellNetwork2Comm > andsLib.h | 2 +- > ShellPkg/DynamicCommand/DpDynamicCommand/Dp.c = | 6 > +++--- > ShellPkg/DynamicCommand/TftpDynamicCommand/Tftp.c = | 6 > +++--- > ShellPkg/Library/UefiHandleParsingLib/UefiHandleParsingLib.c = | 2 +- > ShellPkg/Library/UefiShellBcfgCommandLib/UefiShellBcfgCommandLib.c > | 2 +- > ShellPkg/Library/UefiShellCommandLib/UefiShellCommandLib.c = | 2 > +- >=20 > ShellPkg/Library/UefiShellDebug1CommandsLib/UefiShellDebug1Commands > Lib.c | 2 +- >=20 > ShellPkg/Library/UefiShellDriver1CommandsLib/UefiShellDriver1CommandsL > ib.c | 2 +- >=20 > ShellPkg/Library/UefiShellLevel1CommandsLib/UefiShellLevel1CommandsLib > .c | 2 +- >=20 > ShellPkg/Library/UefiShellLevel2CommandsLib/UefiShellLevel2CommandsLib > .c | 2 +- >=20 > ShellPkg/Library/UefiShellLevel3CommandsLib/UefiShellLevel3CommandsLib > .c | 2 +- > ShellPkg/Library/UefiShellLib/UefiShellLib.c = | 4 ++-- >=20 > ShellPkg/Library/UefiShellNetwork1CommandsLib/UefiShellNetwork1Comm > andsLib.c | 2 +- > ShellPkg/Library/UefiShellNetwork2CommandsLib/UefiShellNetwork2Comm > andsLib.c | 2 +- > 25 files changed, 33 insertions(+), 33 deletions(-) >=20 > diff --git a/ShellPkg/Include/Library/ShellCommandLib.h > b/ShellPkg/Include/Library/ShellCommandLib.h > index 287bc0eba7f9..63fcac82a2de 100644 > --- a/ShellPkg/Include/Library/ShellCommandLib.h > +++ b/ShellPkg/Include/Library/ShellCommandLib.h > @@ -136,7 +136,7 @@ ShellCommandRegisterCommandName ( > IN UINT32 ShellMinSupportLevel, > IN CONST CHAR16 *ProfileName, > IN CONST BOOLEAN CanAffectLE, > - IN CONST EFI_HANDLE HiiHandle, > + IN CONST EFI_HII_HANDLE HiiHandle, > IN CONST EFI_STRING_ID ManFormatHelp > ); >=20 > diff --git a/ShellPkg/Include/Library/ShellLib.h > b/ShellPkg/Include/Library/ShellLib.h > index 31594796cd21..1dc41f2cc11b 100644 > --- a/ShellPkg/Include/Library/ShellLib.h > +++ b/ShellPkg/Include/Library/ShellLib.h > @@ -965,7 +965,7 @@ ShellPrintHiiEx( > IN INT32 Row OPTIONAL, > IN CONST CHAR8 *Language OPTIONAL, > IN CONST EFI_STRING_ID HiiFormatStringId, > - IN CONST EFI_HANDLE HiiFormatHandle, > + IN CONST EFI_HII_HANDLE HiiFormatHandle, > ... > ); >=20 > @@ -1260,7 +1260,7 @@ EFIAPI > ShellPromptForResponseHii ( > IN SHELL_PROMPT_REQUEST_TYPE Type, > IN CONST EFI_STRING_ID HiiFormatStringId, > - IN CONST EFI_HANDLE HiiFormatHandle, > + IN CONST EFI_HII_HANDLE HiiFormatHandle, > IN OUT VOID **Response > ); >=20 > diff --git a/ShellPkg/DynamicCommand/DpDynamicCommand/Dp.h > b/ShellPkg/DynamicCommand/DpDynamicCommand/Dp.h > index 43aa4505ee37..e446cccde923 100644 > --- a/ShellPkg/DynamicCommand/DpDynamicCommand/Dp.h > +++ b/ShellPkg/DynamicCommand/DpDynamicCommand/Dp.h > @@ -36,7 +36,7 @@ > #include #include >=20 > -extern EFI_HANDLE mDpHiiHandle; > +extern EFI_HII_HANDLE mDpHiiHandle; >=20 > #define DP_MAJOR_VERSION 2 > #define DP_MINOR_VERSION 5 > @@ -133,7 +133,7 @@ RunDp ( >=20 > @return HII handle. > **/ > -EFI_HANDLE > +EFI_HII_HANDLE > InitializeHiiPackage ( > EFI_HANDLE ImageHandle > ); > diff --git a/ShellPkg/DynamicCommand/TftpDynamicCommand/Tftp.h > b/ShellPkg/DynamicCommand/TftpDynamicCommand/Tftp.h > index 7a9ed4724e1f..4cd778436813 100644 > --- a/ShellPkg/DynamicCommand/TftpDynamicCommand/Tftp.h > +++ b/ShellPkg/DynamicCommand/TftpDynamicCommand/Tftp.h > @@ -30,7 +30,7 @@ > #include > #include >=20 > -extern EFI_HANDLE mTftpHiiHandle; > +extern EFI_HII_HANDLE mTftpHiiHandle; >=20 > typedef struct { > UINTN FileSize; > @@ -62,7 +62,7 @@ RunTftp ( >=20 > @return HII handle. > **/ > -EFI_HANDLE > +EFI_HII_HANDLE > InitializeHiiPackage ( > EFI_HANDLE ImageHandle > ); > diff --git a/ShellPkg/Library/UefiShellCommandLib/UefiShellCommandLib.h > b/ShellPkg/Library/UefiShellCommandLib/UefiShellCommandLib.h > index 36fe628a8c68..8ecc2f6bf5a2 100644 > --- a/ShellPkg/Library/UefiShellCommandLib/UefiShellCommandLib.h > +++ b/ShellPkg/Library/UefiShellCommandLib/UefiShellCommandLib.h > @@ -46,7 +46,7 @@ typedef struct{ > SHELL_GET_MAN_FILENAME GetManFileName; > SHELL_RUN_COMMAND CommandHandler; > BOOLEAN LastError; > - EFI_HANDLE HiiHandle; > + EFI_HII_HANDLE HiiHandle; > EFI_STRING_ID ManFormatHelp; > } SHELL_COMMAND_INTERNAL_LIST_ENTRY; >=20 > diff --git > a/ShellPkg/Library/UefiShellDebug1CommandsLib/UefiShellDebug1Comman > dsLib.h > b/ShellPkg/Library/UefiShellDebug1CommandsLib/UefiShellDebug1Comman > dsLib.h > index 32a933b9f062..082d488cb283 100644 > --- > a/ShellPkg/Library/UefiShellDebug1CommandsLib/UefiShellDebug1Comman > dsLib.h > +++ > b/ShellPkg/Library/UefiShellDebug1CommandsLib/UefiShellDebug1Comman > d > +++ sLib.h > @@ -52,7 +52,7 @@ > #include >=20 >=20 > -extern EFI_HANDLE gShellDebug1HiiHandle; > +extern EFI_HII_HANDLE gShellDebug1HiiHandle; >=20 > /** > Function returns a system configuration table that is stored in the di= ff --git > a/ShellPkg/Library/UefiShellDriver1CommandsLib/UefiShellDriver1Command > sLib.h > b/ShellPkg/Library/UefiShellDriver1CommandsLib/UefiShellDriver1Comman > dsLib.h > index 7e0b8b094057..ee795c4ce024 100644 > --- > a/ShellPkg/Library/UefiShellDriver1CommandsLib/UefiShellDriver1Command > sLib.h > +++ > b/ShellPkg/Library/UefiShellDriver1CommandsLib/UefiShellDriver1Comma > +++ ndsLib.h > @@ -58,7 +58,7 @@ > #include >=20 >=20 > -extern EFI_HANDLE gShellDriver1HiiHandle; > +extern EFI_HII_HANDLE gShellDriver1HiiHandle; > extern BOOLEAN gInReconnect; >=20 > /** > diff --git > a/ShellPkg/Library/UefiShellLevel1CommandsLib/UefiShellLevel1Commands > Lib.h > b/ShellPkg/Library/UefiShellLevel1CommandsLib/UefiShellLevel1Commands > Lib.h > index 55acdd2b1f95..f2f9cc5dcf3b 100644 > --- > a/ShellPkg/Library/UefiShellLevel1CommandsLib/UefiShellLevel1Commands > Lib.h > +++ > b/ShellPkg/Library/UefiShellLevel1CommandsLib/UefiShellLevel1Command > +++ sLib.h > @@ -33,7 +33,7 @@ > #include > #include >=20 > -extern EFI_HANDLE gShellLevel1HiiHandle; > +extern EFI_HII_HANDLE gShellLevel1HiiHandle; >=20 > /** > Function for 'stall' command. > diff --git > a/ShellPkg/Library/UefiShellLevel2CommandsLib/UefiShellLevel2Commands > Lib.h > b/ShellPkg/Library/UefiShellLevel2CommandsLib/UefiShellLevel2Commands > Lib.h > index 6d522d4bb4a1..77be6f1a12c7 100644 > --- > a/ShellPkg/Library/UefiShellLevel2CommandsLib/UefiShellLevel2Commands > Lib.h > +++ > b/ShellPkg/Library/UefiShellLevel2CommandsLib/UefiShellLevel2Command > +++ sLib.h > @@ -43,7 +43,7 @@ > #include >=20 > extern CONST CHAR16 mFileName[]; > -extern EFI_HANDLE gShellLevel2HiiHandle; > +extern EFI_HII_HANDLE gShellLevel2HiiHandle; >=20 > /** > Function for 'attrib' command. > diff --git > a/ShellPkg/Library/UefiShellLevel3CommandsLib/UefiShellLevel3Commands > Lib.h > b/ShellPkg/Library/UefiShellLevel3CommandsLib/UefiShellLevel3Commands > Lib.h > index 2d97ae4d3c91..c095b9275ed0 100644 > --- > a/ShellPkg/Library/UefiShellLevel3CommandsLib/UefiShellLevel3Commands > Lib.h > +++ > b/ShellPkg/Library/UefiShellLevel3CommandsLib/UefiShellLevel3Command > +++ sLib.h > @@ -32,7 +32,7 @@ > #include > #include >=20 > -extern EFI_HANDLE gShellLevel3HiiHandle; > +extern EFI_HII_HANDLE gShellLevel3HiiHandle; >=20 > /** > Function for 'type' command. > diff --git > a/ShellPkg/Library/UefiShellNetwork1CommandsLib/UefiShellNetwork1Com > mandsLib.h > b/ShellPkg/Library/UefiShellNetwork1CommandsLib/UefiShellNetwork1Com > mandsLib.h > index d4ed8c04652d..fddada2efa48 100644 > --- > a/ShellPkg/Library/UefiShellNetwork1CommandsLib/UefiShellNetwork1Com > mandsLib.h > +++ > b/ShellPkg/Library/UefiShellNetwork1CommandsLib/UefiShellNetwork1Com > +++ mandsLib.h > @@ -38,7 +38,7 @@ > #include > #include >=20 > -extern EFI_HANDLE gShellNetwork1HiiHandle; > +extern EFI_HII_HANDLE gShellNetwork1HiiHandle; >=20 > /** > Function for 'ping' command. > diff --git > a/ShellPkg/Library/UefiShellNetwork2CommandsLib/UefiShellNetwork2Com > mandsLib.h > b/ShellPkg/Library/UefiShellNetwork2CommandsLib/UefiShellNetwork2Com > mandsLib.h > index 9a5db32f2b76..9ea42cf26d53 100644 > --- > a/ShellPkg/Library/UefiShellNetwork2CommandsLib/UefiShellNetwork2Com > mandsLib.h > +++ > b/ShellPkg/Library/UefiShellNetwork2CommandsLib/UefiShellNetwork2Com > +++ mandsLib.h > @@ -27,7 +27,7 @@ > #include > #include >=20 > -extern EFI_HANDLE gShellNetwork2HiiHandle; > +extern EFI_HII_HANDLE gShellNetwork2HiiHandle; >=20 > /** > Function for 'ping6' command. > diff --git a/ShellPkg/DynamicCommand/DpDynamicCommand/Dp.c > b/ShellPkg/DynamicCommand/DpDynamicCommand/Dp.c > index 735cdcbcc018..4ec4c18348bd 100644 > --- a/ShellPkg/DynamicCommand/DpDynamicCommand/Dp.c > +++ b/ShellPkg/DynamicCommand/DpDynamicCommand/Dp.c > @@ -36,7 +36,7 @@ typedef struct { >=20 > #pragma pack() >=20 > -EFI_HANDLE mDpHiiHandle; > +EFI_HII_HANDLE mDpHiiHandle; >=20 > typedef struct { > EFI_HANDLE Handle; > @@ -924,14 +924,14 @@ Done: >=20 > @return HII handle. > **/ > -EFI_HANDLE > +EFI_HII_HANDLE > InitializeHiiPackage ( > EFI_HANDLE ImageHandle > ) > { > EFI_STATUS Status; > EFI_HII_PACKAGE_LIST_HEADER *PackageList; > - EFI_HANDLE HiiHandle; > + EFI_HII_HANDLE HiiHandle; >=20 > // > // Retrieve HII package list from ImageHandle diff --git > a/ShellPkg/DynamicCommand/TftpDynamicCommand/Tftp.c > b/ShellPkg/DynamicCommand/TftpDynamicCommand/Tftp.c > index 607899032e9d..f28da9af723c 100644 > --- a/ShellPkg/DynamicCommand/TftpDynamicCommand/Tftp.c > +++ b/ShellPkg/DynamicCommand/TftpDynamicCommand/Tftp.c > @@ -11,7 +11,7 @@ > #include "Tftp.h" >=20 > #define IP4_CONFIG2_INTERFACE_INFO_NAME_LENGTH 32 > -EFI_HANDLE mTftpHiiHandle; > +EFI_HII_HANDLE mTftpHiiHandle; >=20 > /* > Constant strings and definitions related to the message indicating th= e > amount of @@ -1087,14 +1087,14 @@ CheckPacket ( >=20 > @return HII handle. > **/ > -EFI_HANDLE > +EFI_HII_HANDLE > InitializeHiiPackage ( > EFI_HANDLE ImageHandle > ) > { > EFI_STATUS Status; > EFI_HII_PACKAGE_LIST_HEADER *PackageList; > - EFI_HANDLE HiiHandle; > + EFI_HII_HANDLE HiiHandle; >=20 > // > // Retrieve HII package list from ImageHandle diff --git > a/ShellPkg/Library/UefiHandleParsingLib/UefiHandleParsingLib.c > b/ShellPkg/Library/UefiHandleParsingLib/UefiHandleParsingLib.c > index f179c4109223..f62d30ef677a 100644 > --- a/ShellPkg/Library/UefiHandleParsingLib/UefiHandleParsingLib.c > +++ b/ShellPkg/Library/UefiHandleParsingLib/UefiHandleParsingLib.c > @@ -14,7 +14,7 @@ > #include > #include >=20 > -EFI_HANDLE mHandleParsingHiiHandle =3D NULL; > +EFI_HII_HANDLE mHandleParsingHiiHandle =3D NULL; > HANDLE_INDEX_LIST mHandleList =3D {{{NULL,NULL},0,0},0}; > GUID_INFO_BLOCK *mGuidList; > UINTN mGuidListCount; > diff --git > a/ShellPkg/Library/UefiShellBcfgCommandLib/UefiShellBcfgCommandLib.c > b/ShellPkg/Library/UefiShellBcfgCommandLib/UefiShellBcfgCommandLib.c > index e8b48b4990dd..f8bcaebe46c8 100644 > --- > a/ShellPkg/Library/UefiShellBcfgCommandLib/UefiShellBcfgCommandLib.c > +++ > b/ShellPkg/Library/UefiShellBcfgCommandLib/UefiShellBcfgCommandLib.c > @@ -38,7 +38,7 @@ > #include >=20 > STATIC CONST CHAR16 mFileName[] =3D L"ShellCommands"; -STATIC > EFI_HANDLE gShellBcfgHiiHandle =3D NULL; > +STATIC EFI_HII_HANDLE gShellBcfgHiiHandle =3D NULL; >=20 > typedef enum { > BcfgTargetBootOrder =3D 0, > diff --git a/ShellPkg/Library/UefiShellCommandLib/UefiShellCommandLib.c > b/ShellPkg/Library/UefiShellCommandLib/UefiShellCommandLib.c > index 826ced30a8c8..4c48b65fbc1d 100644 > --- a/ShellPkg/Library/UefiShellCommandLib/UefiShellCommandLib.c > +++ b/ShellPkg/Library/UefiShellCommandLib/UefiShellCommandLib.c > @@ -554,7 +554,7 @@ ShellCommandRegisterCommandName ( > IN UINT32 ShellMinSupportLevel, > IN CONST CHAR16 *ProfileName, > IN CONST BOOLEAN CanAffectLE, > - IN CONST EFI_HANDLE HiiHandle, > + IN CONST EFI_HII_HANDLE HiiHandle, > IN CONST EFI_STRING_ID ManFormatHelp > ) > { > diff --git > a/ShellPkg/Library/UefiShellDebug1CommandsLib/UefiShellDebug1Comman > dsLib.c > b/ShellPkg/Library/UefiShellDebug1CommandsLib/UefiShellDebug1Comman > dsLib.c > index ddce3bef5a30..f918867f47af 100644 > --- > a/ShellPkg/Library/UefiShellDebug1CommandsLib/UefiShellDebug1Comman > dsLib.c > +++ > b/ShellPkg/Library/UefiShellDebug1CommandsLib/UefiShellDebug1Comman > d > +++ sLib.c > @@ -10,7 +10,7 @@ > #include >=20 > STATIC CONST CHAR16 mFileName[] =3D L"Debug1Commands"; -EFI_HANDLE > gShellDebug1HiiHandle =3D NULL; > +EFI_HII_HANDLE gShellDebug1HiiHandle =3D NULL; >=20 > /** > Gets the debug file name. This will be used if HII is not working. > diff --git > a/ShellPkg/Library/UefiShellDriver1CommandsLib/UefiShellDriver1Command > sLib.c > b/ShellPkg/Library/UefiShellDriver1CommandsLib/UefiShellDriver1Comman > dsLib.c > index 4a05fa9942c4..e2219c62ec25 100644 > --- > a/ShellPkg/Library/UefiShellDriver1CommandsLib/UefiShellDriver1Command > sLib.c > +++ > b/ShellPkg/Library/UefiShellDriver1CommandsLib/UefiShellDriver1Comma > +++ ndsLib.c > @@ -9,7 +9,7 @@ > #include "UefiShellDriver1CommandsLib.h" >=20 > STATIC CONST CHAR16 mFileName[] =3D L"Driver1Commands"; -EFI_HANDLE > gShellDriver1HiiHandle =3D NULL; > +EFI_HII_HANDLE gShellDriver1HiiHandle =3D NULL; > BOOLEAN gInReconnect =3D FALSE; >=20 > /** > diff --git > a/ShellPkg/Library/UefiShellLevel1CommandsLib/UefiShellLevel1Commands > Lib.c > b/ShellPkg/Library/UefiShellLevel1CommandsLib/UefiShellLevel1Commands > Lib.c > index ecbee99e3b3d..88cddd88ddc4 100644 > --- > a/ShellPkg/Library/UefiShellLevel1CommandsLib/UefiShellLevel1Commands > Lib.c > +++ > b/ShellPkg/Library/UefiShellLevel1CommandsLib/UefiShellLevel1Command > +++ sLib.c > @@ -10,7 +10,7 @@ > #include "UefiShellLevel1CommandsLib.h" >=20 > STATIC CONST CHAR16 mFileName[] =3D L"ShellCommands"; -EFI_HANDLE > gShellLevel1HiiHandle =3D NULL; > +EFI_HII_HANDLE gShellLevel1HiiHandle =3D NULL; >=20 > /** > Return the help text filename. Only used if no HII information found. > diff --git > a/ShellPkg/Library/UefiShellLevel2CommandsLib/UefiShellLevel2Commands > Lib.c > b/ShellPkg/Library/UefiShellLevel2CommandsLib/UefiShellLevel2Commands > Lib.c > index c2a0bb492fbb..69427637bb87 100644 > --- > a/ShellPkg/Library/UefiShellLevel2CommandsLib/UefiShellLevel2Commands > Lib.c > +++ > b/ShellPkg/Library/UefiShellLevel2CommandsLib/UefiShellLevel2Command > +++ sLib.c > @@ -29,7 +29,7 @@ > #include "UefiShellLevel2CommandsLib.h" >=20 > CONST CHAR16 mFileName[] =3D L"ShellCommands"; -EFI_HANDLE > gShellLevel2HiiHandle =3D NULL; > +EFI_HII_HANDLE gShellLevel2HiiHandle =3D NULL; >=20 > /** > Get the filename to get help text from if not using HII. > diff --git > a/ShellPkg/Library/UefiShellLevel3CommandsLib/UefiShellLevel3Commands > Lib.c > b/ShellPkg/Library/UefiShellLevel3CommandsLib/UefiShellLevel3Commands > Lib.c > index 7d2cc4a48371..ce4afd117aa1 100644 > --- > a/ShellPkg/Library/UefiShellLevel3CommandsLib/UefiShellLevel3Commands > Lib.c > +++ > b/ShellPkg/Library/UefiShellLevel3CommandsLib/UefiShellLevel3Command > +++ sLib.c > @@ -9,7 +9,7 @@ > #include "UefiShellLevel3CommandsLib.h" >=20 > CONST CHAR16 gShellLevel3FileName[] =3D L"ShellCommands"; -EFI_HANDLE > gShellLevel3HiiHandle =3D NULL; > +EFI_HII_HANDLE gShellLevel3HiiHandle =3D NULL; >=20 > /** > return the filename to get help from is not using HII. > diff --git a/ShellPkg/Library/UefiShellLib/UefiShellLib.c > b/ShellPkg/Library/UefiShellLib/UefiShellLib.c > index 5be530092e1b..835d0f88ca74 100644 > --- a/ShellPkg/Library/UefiShellLib/UefiShellLib.c > +++ b/ShellPkg/Library/UefiShellLib/UefiShellLib.c > @@ -2997,7 +2997,7 @@ ShellPrintHiiEx( > IN INT32 Row OPTIONAL, > IN CONST CHAR8 *Language OPTIONAL, > IN CONST EFI_STRING_ID HiiFormatStringId, > - IN CONST EFI_HANDLE HiiFormatHandle, > + IN CONST EFI_HII_HANDLE HiiFormatHandle, > ... > ) > { > @@ -3609,7 +3609,7 @@ EFIAPI > ShellPromptForResponseHii ( > IN SHELL_PROMPT_REQUEST_TYPE Type, > IN CONST EFI_STRING_ID HiiFormatStringId, > - IN CONST EFI_HANDLE HiiFormatHandle, > + IN CONST EFI_HII_HANDLE HiiFormatHandle, > IN OUT VOID **Response > ) > { > diff --git > a/ShellPkg/Library/UefiShellNetwork1CommandsLib/UefiShellNetwork1Com > mandsLib.c > b/ShellPkg/Library/UefiShellNetwork1CommandsLib/UefiShellNetwork1Com > mandsLib.c > index 7e823cabd2fe..9a2b23fdc5ba 100644 > --- > a/ShellPkg/Library/UefiShellNetwork1CommandsLib/UefiShellNetwork1Com > mandsLib.c > +++ > b/ShellPkg/Library/UefiShellNetwork1CommandsLib/UefiShellNetwork1Com > +++ mandsLib.c > @@ -8,7 +8,7 @@ > #include "UefiShellNetwork1CommandsLib.h" >=20 > CONST CHAR16 gShellNetwork1FileName[] =3D L"ShellCommands"; - > EFI_HANDLE gShellNetwork1HiiHandle =3D NULL; > +EFI_HII_HANDLE gShellNetwork1HiiHandle =3D NULL; >=20 > /** > return the file name of the help text file if not using HII. > diff --git > a/ShellPkg/Library/UefiShellNetwork2CommandsLib/UefiShellNetwork2Com > mandsLib.c > b/ShellPkg/Library/UefiShellNetwork2CommandsLib/UefiShellNetwork2Com > mandsLib.c > index 5a7ffbfa19eb..4aab4295c1ba 100644 > --- > a/ShellPkg/Library/UefiShellNetwork2CommandsLib/UefiShellNetwork2Com > mandsLib.c > +++ > b/ShellPkg/Library/UefiShellNetwork2CommandsLib/UefiShellNetwork2Com > +++ mandsLib.c > @@ -8,7 +8,7 @@ > #include "UefiShellNetwork2CommandsLib.h" >=20 > CONST CHAR16 gShellNetwork2FileName[] =3D L"ShellCommands"; - > EFI_HANDLE gShellNetwork2HiiHandle =3D NULL; > +EFI_HII_HANDLE gShellNetwork2HiiHandle =3D NULL; >=20 > /** > return the file name of the help text file if not using HII. > -- > 2.19.1.3.g30247aa5d201 >=20