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

[Xiang, Haihao]

Thanks for your testing,  the patch was merged into master.

Regards
Haihao


> -----Original Message-----
> From: Charles, Daniel [mailto:daniel.charles at intel.com]
> Sent: Thursday, June 26, 2014 6:10 AM
> To: Xiang, Haihao
> Cc: libva at lists.freedesktop.org
> Subject: Re: [Libva] [PATCH] va: release VADisplayContext at the end of
> vaTerminate()
> 
> 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