[Bug 783255] adaptivedemux: release the manifest lock ...

Thu Jun 1 13:35:02 UTC 2017

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

Edward Hervey <bilboed at bilboed.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |RESOLVED
                 CC|                            |bilboed at bilboed.com
         Resolution|---                         |FIXED

--- Comment #5 from Edward Hervey <bilboed at bilboed.com> ---
commit 694da006e041cfd3fa1247525f21fdaedebac446
Author: Mathieu Duponchelle <mathieu.duponchelle at opencreed.com>
Date:   Wed May 31 02:46:01 2017 +0200

    adaptivedemux: release the manifest lock ...

    before broadcasting preroll.

    The deadlock was as follows:

    -> The subclass pushes a buffer on a newly-created stream in T1
    -> We take the preroll lock in T1, to handle_preroll
    -> The demuxer is stopped in T2, we take the MANIFEST_LOCK
    -> T1 starts blocking because it received a reconfigure event
       and needs to take the MANIFEST_LOCK
    -> T2 deadlocks because it now wants the preroll_lock.

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

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