[Bug 785957] adaptivedemux: leaks pad probes, causing gradual increase in CPU

[GStreamer (GNOME Bugzilla)]

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

Tim-Philipp Müller <t.i.m at zen.co.uk> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |RESOLVED
         Resolution|---                         |FIXED
   Target Milestone|git master                  |1.12.3
            Summary|GstAdaptiveDemuxStream      |adaptivedemux: leaks pad
                   |leaks pad probes, causing   |probes, causing gradual
                   |gradual increase in CPU     |increase in CPU

--- Comment #4 from Tim-Philipp Müller <t.i.m at zen.co.uk> ---
Many thanks for the patch, pushed to master and 1.12 branch:

commit 4064683d36ba8eb8732b904be0bdaedc38b013fc
Author: Tom Bailey <tom.bailey at youview.com>
Date:   Wed Aug 2 18:17:08 2017 +0100

    adaptivedemux: Fix leak of pad probes in GstAdaptiveDemuxStream

    This commit ensures that the idle probe which GstAdaptiveDemuxStream
    adds to the upstream source pad is removed after use. Previously a new
    probe was added to the pad whenever a fragment was downloaded, meaning
    the number of pad probe callbacks being executed increased continually.

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

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