[Bug 790752] msdk: supports bufferpool

[GStreamer (GNOME Bugzilla)]

https://bugzilla.gnome.org/show_bug.cgi?id=790752

--- Comment #86 from Hyunjun Ko <zzoon at igalia.com> ---
(In reply to sreerenj from comment #79)
> Review of attachment 368112 [details] [review]:
> 
> ::: sys/msdk/gstmsdkallocator_libva.c
> @@ +159,3 @@
> +
> +  if (mem_id->fourcc != MFX_FOURCC_P8) {
> +    va_status =
> 
> I think you need to make sure the Surface is not mapped.
> Or at least the vaDestoryImage is required to make sure the VAImage
> Structure is not leaking. right?

Good point. I would simply call vaDestroyImage for each surface.

-- 
You are receiving this mail because:
You are the QA Contact for the bug.
You are the assignee for the bug.