[Mesa-dev] [st/va] Fix leak in VAAPI subpictures

[Alex Deucher]

On Thu, Jul 6, 2017 at 2:37 PM, Andres Gomez <agomez at igalia.com> wrote:
> It looks like we could want this into -stable (?)

Yes, I think so.

Alex

>
> On Thu, 2017-07-06 at 21:10 +0300, Andres Gomez wrote:
>> On Thu, 2017-06-22 at 09:25 +0000, Namburu, Chandu-babu wrote:
>> > From: Chandu Babu N <chandu at amd.com>
>> > Subject: [PATCH] [st/va] Fix leak in VAAPI subpictures
>> >
>> > sampler view allocated in vaAssociateSubpicture is not cleared
>> > in vaiDeassociateSubpicture.
>> >
>> > Reviewed-by: Christian König <christian.koenig at amd.com>
>> > ---
>> > src/gallium/state_trackers/va/subpicture.c | 1 +
>> > 1 file changed, 1 insertion(+)
>> >
>> > diff --git a/src/gallium/state_trackers/va/subpicture.c b/src/gallium/state_trackers/va/subpicture.c
>> > index 15d52b9..981a99c 100644
>> > --- a/src/gallium/state_trackers/va/subpicture.c
>> > +++ b/src/gallium/state_trackers/va/subpicture.c
>> > @@ -283,6 +283,7 @@ vlVaDeassociateSubpicture(VADriverContextP ctx, VASubpictureID subpicture,
>> >        while (surf->subpics.size && util_dynarray_top(&surf->subpics, vlVaSubpicture *) == NULL)
>> >           (void)util_dynarray_pop(&surf->subpics, vlVaSubpicture *);
>> >     }
>> > +   pipe_sampler_view_reference(&sub->sampler,NULL);
>> >     mtx_unlock(&drv->mutex);
>> >     return VA_STATUS_SUCCESS;
>> > --
>> > 2.7.4
>> > _______________________________________________
>> > mesa-dev mailing list
>> > mesa-dev at lists.freedesktop.org
>> > https://lists.freedesktop.org/mailman/listinfo/mesa-dev
> --
> Br,
>
> Andres
> _______________________________________________
> mesa-dev mailing list
> mesa-dev at lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/mesa-dev