IntelFrameworkModulePkg/LegacyBootMaintUiLib: Add NULL pointer check
mLegacyBootOptionPrivate pointer is initialized in Constructor function with if condition check, but it's used in Destructor function directly without any check. Now add the NULL pointer check. Cc: Eric Dong <eric.dong@intel.com> Cc: Liming Gao <liming.gao@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>
This commit is contained in:
@@ -15,7 +15,7 @@ WITHOUT WARRANTIES OR REPRESENTATIONS OF ANY KIND, EITHER EXPRESS OR IMPLIED.
|
||||
|
||||
#include "LegacyBootMaintUi.h"
|
||||
|
||||
LEGACY_BOOT_OPTION_CALLBACK_DATA *mLegacyBootOptionPrivate;
|
||||
LEGACY_BOOT_OPTION_CALLBACK_DATA *mLegacyBootOptionPrivate = NULL;
|
||||
EFI_GUID mLegacyBootOptionGuid = LEGACY_BOOT_OPTION_FORMSET_GUID;
|
||||
CHAR16 mLegacyBootStorageName[] = L"LegacyBootData";
|
||||
BBS_TYPE mBbsType[] = {BBS_FLOPPY, BBS_HARDDISK, BBS_CDROM, BBS_EMBED_NETWORK, BBS_BEV_DEVICE, BBS_UNKNOWN};
|
||||
@@ -1486,7 +1486,7 @@ LegacyBootMaintUiLibDestructor (
|
||||
{
|
||||
EFI_STATUS Status;
|
||||
|
||||
if (mLegacyBootOptionPrivate->DriverHandle != NULL) {
|
||||
if (mLegacyBootOptionPrivate != NULL && mLegacyBootOptionPrivate->DriverHandle != NULL) {
|
||||
Status = gBS->UninstallMultipleProtocolInterfaces (
|
||||
mLegacyBootOptionPrivate->DriverHandle,
|
||||
&gEfiDevicePathProtocolGuid,
|
||||
|
Reference in New Issue
Block a user