[Libva] [PATCH] va: release VADisplayContext at the end of vaTerminate()

Charles, Daniel daniel.charles at intel.com
Wed Jun 25 15:09:44 PDT 2014


On Wed, Jul 17, 2013 at 10:21 PM, Xiang, Haihao <haihao.xiang at intel.com> wrote:
> From: "Xiang, Haihao" <haihao.xiang at intel.com>
>
> Otherwise it will result in invalid reading in va_TraceEnd()
>
> Signed-off-by: Xiang, Haihao <haihao.xiang at intel.com>
> ---
>  va/va.c |    6 +++---
>  1 file changed, 3 insertions(+), 3 deletions(-)
>
> diff --git a/va/va.c b/va/va.c
> index 6cb17ef..0ba595b 100644
> --- a/va/va.c
> +++ b/va/va.c
> @@ -515,15 +515,15 @@ VAStatus vaTerminate (
>    free(old_ctx->vtable_vpp);
>    old_ctx->vtable_vpp = NULL;
>
> -  if (VA_STATUS_SUCCESS == vaStatus)
> -      pDisplayContext->vaDestroy(pDisplayContext);
> -
>    VA_TRACE_LOG(va_TraceTerminate, dpy);
>
>    va_TraceEnd(dpy);
>
>    va_FoolEnd(dpy);
>
> +  if (VA_STATUS_SUCCESS == vaStatus)
> +      pDisplayContext->vaDestroy(pDisplayContext);
> +
>    return vaStatus;
>  }
>

Tested-by: Daniel Charles <daniel.charles at intel.com> on ChromiumOS

Please merge to master

Thanks.

-- 
Daniel.
> --
> 1.7.9.5
>
> _______________________________________________
> Libva mailing list
> Libva at lists.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/libva


More information about the Libva mailing list