On Tue, Nov 19, 2013 at 10:36 AM, Laszlo Ersek <ler...@redhat.com> wrote:
> Contributed-under: TianoCore Contribution Agreement 1.0
> Signed-off-by: Laszlo Ersek <ler...@redhat.com>
> ---
>  OvmfPkg/QemuVideoDxe/Driver.c | 4 ++++
>  OvmfPkg/QemuVideoDxe/Gop.c    | 1 +
>  2 files changed, 5 insertions(+)
>
> diff --git a/OvmfPkg/QemuVideoDxe/Driver.c b/OvmfPkg/QemuVideoDxe/Driver.c
> index 1dd8899..53718e1 100644
> --- a/OvmfPkg/QemuVideoDxe/Driver.c
> +++ b/OvmfPkg/QemuVideoDxe/Driver.c
> @@ -305,6 +305,10 @@ QemuVideoControllerDriverStart (
>        DEBUG ((EFI_D_INFO, "QemuVideo: Using mmio bar @ 0x%lx\n",
>                MmioDesc->AddrRangeMin));
>      }
> +
> +    if (!EFI_ERROR (Status)) {
> +      FreePool (MmioDesc);
> +    }
>    }
>
>    //
> diff --git a/OvmfPkg/QemuVideoDxe/Gop.c b/OvmfPkg/QemuVideoDxe/Gop.c
> index 1d2402c..30aac7f 100644
> --- a/OvmfPkg/QemuVideoDxe/Gop.c
> +++ b/OvmfPkg/QemuVideoDxe/Gop.c
> @@ -72,6 +72,7 @@ QemuVideoCompleteModeData (
>    Mode->FrameBufferSize = Mode->FrameBufferSize * ((ModeData->ColorDepth + 
> 7) / 8);
>    DEBUG ((EFI_D_INFO, "FrameBufferBase: 0x%x, FrameBufferSize: 0x%x\n", 
> Mode->FrameBufferBase, Mode->FrameBufferSize));
>
> +  FreePool (FrameBufDesc);

Seems to have been inherited from
CirrusLogic5430Dxe/CirrusLogic5430GraphicsOutput.c.

Reviewed-by: Jordan Justen <jordan.l.jus...@intel.com>

-Jordan

------------------------------------------------------------------------------
Shape the Mobile Experience: Free Subscription
Software experts and developers: Be at the forefront of tech innovation.
Intel(R) Software Adrenaline delivers strategic insight and game-changing 
conversations that shape the rapidly evolving mobile landscape. Sign up now. 
http://pubads.g.doubleclick.net/gampad/clk?id=63431311&iu=/4140/ostg.clktrk
_______________________________________________
edk2-devel mailing list
edk2-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/edk2-devel

Reply via email to