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

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

Reviewed-by: Liming Gao <liming.gao@intel.com>

> -----Original Message-----
> From: Song, BinX
> Sent: Thursday, November 10, 2016 4:20 PM
> To: edk2-devel@lists.01.org
> Cc: Gao, Liming <liming.gao@intel.com>
> Subject: [PATCH V2] MdeModulePkg/BootLogoLib: Remove invalid if
> judgments
> 
> FreePool (Blt) function will be called in while loop, cannot be removed.
> 
> 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 | 4 ----
>  1 file changed, 4 deletions(-)
> 
> diff --git a/MdeModulePkg/Library/BootLogoLib/BootLogoLib.c
> b/MdeModulePkg/Library/BootLogoLib/BootLogoLib.c
> index 2c1e8ea..8bd9985 100644
> --- a/MdeModulePkg/Library/BootLogoLib/BootLogoLib.c
> +++ b/MdeModulePkg/Library/BootLogoLib/BootLogoLib.c
> @@ -142,10 +142,6 @@ BootLogoEnableLogo (
>        break;
>      }
> 
> -    if (EFI_ERROR (Status)) {
> -      continue;
> -    }
> -
>      if (Blt != NULL) {
>        FreePool (Blt);
>      }
> --
> 2.10.2.windows.1