[Bug 702705] rtspsrc does not pause properly (race condition)

[GStreamer (bugzilla.gnome.org)]

https://bugzilla.gnome.org/show_bug.cgi?id=702705
  GStreamer | gst-plugins-good | git

Wim Taymans <wim.taymans> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|UNCONFIRMED                 |RESOLVED
                 CC|                            |wim.taymans at gmail.com
         Resolution|                            |FIXED
   Target Milestone|HEAD                        |1.0.8

--- Comment #1 from Wim Taymans <wim.taymans at gmail.com> 2013-06-20 12:49:00 UTC ---
Your analysis is spot on and the proposed solution can be simplified by simply
taking and releasing the stream lock, like how the seek does it.

I don't quite understand what the second patch does, it is needed? when is it
needed?

commit b96d931bf4ce5c3c8c5be4fa76f5c75bca85d0c4
Author: Wim Taymans <wim.taymans at collabora.co.uk>
Date:   Thu Jun 20 14:43:47 2013 +0200

    rtspsrc: fix race in state change to paused

    When we go to paused, we first flush the connection and then send the pause
    command. As a result of the flushing, the scheduled paused command can get
    lost. Wait until the connection is completely flushed and the rtsp task is
    waiting before issuing the paused or playing request.

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

-- 
Configure bugmail: https://bugzilla.gnome.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the QA contact for the bug.
You are the assignee for the bug.