Re: [PATCH] MdeModulePkg/BootLogoLib: Remove invalid if judgments

["Gao, Liming" <liming.gao@intel.com>]

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.gao@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.gao@intel.com>
> Contributed-under: TianoCore Contribution Agreement 1.0
> Signed-off-by: Bell Song <binx.song@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