Reviewed-by: Dandan Bi <dandan...@intel.com>
Thanks, Dandan > -----Original Message----- > From: Nickle Wang [mailto:nickle.w...@hpe.com] > Sent: Friday, February 21, 2020 9:57 AM > To: devel@edk2.groups.io; nickle.w...@hpe.com > Cc: Bi, Dandan <dandan...@intel.com> > Subject: [PATCH v2] MdeModulePkg/SetupBrowserDxe: Fix IsZeroGuid() ASSERT. > > From the function description of GetIfrBinaryData(), FormSetGuid can be NULL. > However, FormSetGuid is passed to IsZeroGuid(). This causes exception when > FormSetGuid is NULL. > > Signed-off-by: Nickle Wang <nickle.w...@hpe.com> > --- > MdeModulePkg/Universal/SetupBrowserDxe/Setup.c | 3 ++- > 1 file changed, 2 insertions(+), 1 deletion(-) > > diff --git a/MdeModulePkg/Universal/SetupBrowserDxe/Setup.c > b/MdeModulePkg/Universal/SetupBrowserDxe/Setup.c > index 288f1c3197..82067b541c 100644 > --- a/MdeModulePkg/Universal/SetupBrowserDxe/Setup.c > +++ b/MdeModulePkg/Universal/SetupBrowserDxe/Setup.c > @@ -2,6 +2,7 @@ > Entry and initialization module for the browser. > > Copyright (c) 2007 - 2018, Intel Corporation. All rights reserved.<BR> > +(C) Copyright 2020 Hewlett Packard Enterprise Development LP<BR> > SPDX-License-Identifier: BSD-2-Clause-Patent > > **/ > @@ -5844,7 +5845,7 @@ GetIfrBinaryData ( > // > // Try to compare against formset GUID > // > - if (IsZeroGuid (FormSetGuid) || > + if (IsZeroGuid (ComparingGuid) || > CompareGuid (ComparingGuid, (EFI_GUID *)(OpCodeData + sizeof > (EFI_IFR_OP_HEADER)))) { > break; > } > -- > 2.20.1.windows.1 -=-=-=-=-=-=-=-=-=-=-=- Groups.io Links: You receive all messages sent to this group. View/Reply Online (#54698): https://edk2.groups.io/g/devel/message/54698 Mute This Topic: https://groups.io/mt/71442938/21656 Group Owner: devel+ow...@edk2.groups.io Unsubscribe: https://edk2.groups.io/g/devel/unsub [arch...@mail-archive.com] -=-=-=-=-=-=-=-=-=-=-=-
