[Bug 767934] vaapidecoder_h265: critical memory leak

[GStreamer (GNOME Bugzilla)]

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

Víctor Manuel Jáquez Leal <vjaquez at igalia.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
 Attachment #330174|none                        |reviewed
             status|                            |

--- Comment #2 from Víctor Manuel Jáquez Leal <vjaquez at igalia.com> ---
Review of attachment 330174:
 --> (https://bugzilla.gnome.org/review?bug=767934&attachment=330174)

H264 and H265 decoders are very similar, and in H264 is not failing, if I
understand correctly. Why is happening only in H265?

::: gst-libs/gst/vaapi/gstvaapidecoder_h265.c
@@ -747,3 @@
   if (hard_flush) {
-    for (i = 0; i < priv->dpb_count; i++)
-      dpb_remove_index (decoder, i);

The question is, why dpb_remove_index() is not working as expected? why do we
need another function?

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