[Bug 756611] playbin: Leak of playbin on errors from the source element

[GStreamer (GNOME Bugzilla)]

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

Sebastian Dröge (slomo) <slomo at coaxion.net> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |RESOLVED
         Resolution|---                         |FIXED
   Target Milestone|git master                  |1.7.1

--- Comment #21 from Sebastian Dröge (slomo) <slomo at coaxion.net> ---
Will go to 1.6.1 in a bit, thanks for testing Vineeth! :)

commit 2955eeb0eda1c87fb323368c86ca6456c4da522d
Author: Sebastian Dröge <sebastian at centricular.com>
Date:   Sat Oct 17 22:13:08 2015 +0300

    bin: Make sure to free all cached messages when going to NULL

    An ASYNC READY->PAUSED might have failed without the bin code noticing
during
    the state change, in which case we will never get PAUSED->READY and would
leak
    messages.

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



commit 4d6aa0f83121c7c19561124ccff0f59faa4dca97
Author: Sebastian Dröge <sebastian at centricular.com>
Date:   Sat Oct 17 22:25:22 2015 +0300

    decodebin/playbin/playsink/subtitleoverlay: Post async-done on state change
failures

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

commit 87dbe54797d7e5e1d89f3549d9b888533482496d
Author: Sebastian Dröge <sebastian at centricular.com>
Date:   Sat Oct 17 22:20:31 2015 +0300

    playsink: Immediately error out if state change fails

    Otherwise we chain up to the parent class' change_state function and might
    override the failure with SUCCESS.

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

commit 92061cb19e81e5846628dfb032e29defa917413e
Author: Sebastian Dröge <sebastian at centricular.com>
Date:   Sat Oct 17 21:47:07 2015 +0300

    playbin/uridecodebin: Always post async-done immediately if we're a live
pipeline

    Not only if the base class told us, but also if one of our own elements
did.

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

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