Bin: Why remove FreePool (Blt)? It is still used. Thanks Liming > -----Original Message----- > From: Song, BinX > Sent: Tuesday, November 08, 2016 11:01 AM > To: edk2-devel@lists.01.org > Cc: Gao, Liming <liming....@intel.com> > Subject: [PATCH] MdeModulePkg/BootLogoLib: Remove invalid if judgments > > There are two invalid if judgments in BootLogoEnableLogo() function, > remove them. > > Cc: Liming Gao <liming....@intel.com> > Contributed-under: TianoCore Contribution Agreement 1.0 > Signed-off-by: Bell Song <binx.s...@intel.com> > --- > MdeModulePkg/Library/BootLogoLib/BootLogoLib.c | 8 -------- > 1 file changed, 8 deletions(-) > > diff --git a/MdeModulePkg/Library/BootLogoLib/BootLogoLib.c > b/MdeModulePkg/Library/BootLogoLib/BootLogoLib.c > index 2c1e8ea..b69dda8 100644 > --- a/MdeModulePkg/Library/BootLogoLib/BootLogoLib.c > +++ b/MdeModulePkg/Library/BootLogoLib/BootLogoLib.c > @@ -141,14 +141,6 @@ BootLogoEnableLogo ( > if (EFI_ERROR (Status)) { > break; > } > - > - if (EFI_ERROR (Status)) { > - continue; > - } > - > - if (Blt != NULL) { > - FreePool (Blt); > - } > Blt = Image.Bitmap; > > // > -- > 2.7.2.windows.1
_______________________________________________ edk2-devel mailing list edk2-devel@lists.01.org https://lists.01.org/mailman/listinfo/edk2-devel