[Bug 783256] adaptivedemux: make sure to free all "old streams"

GStreamer (GNOME Bugzilla) bugzilla at gnome.org
Wed May 31 09:41:38 UTC 2017


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

Edward Hervey <bilboed at bilboed.com> changed:

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

--- Comment #3 from Edward Hervey <bilboed at bilboed.com> ---
Review of attachment 352912:
 --> (https://bugzilla.gnome.org/review?bug=783256&attachment=352912)

::: gst-libs/gst/adaptivedemux/gstadaptivedemux.c
@@ +809,3 @@
     g_list_free_full (old_streams,
         (GDestroyNotify) gst_adaptive_demux_stream_free);
+  } else if (demux->priv->old_streams) {

why "else if" and not just "if" ?

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


More information about the gstreamer-bugs mailing list