[Bug 694184] New: Wrong code organisation in case of collision in rtpsession

[GStreamer (bugzilla.gnome.org)]

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

           Summary: Wrong code organisation in case of collision in
                    rtpsession
    Classification: Platform
           Product: GStreamer
           Version: 1.0.5
        OS/Version: Linux
            Status: UNCONFIRMED
          Severity: normal
          Priority: Normal
         Component: gst-plugins-good
        AssignedTo: gstreamer-bugs at lists.freedesktop.org
        ReportedBy: visechelle at gmail.com
         QAContact: gstreamer-bugs at lists.freedesktop.org
     GNOME version: ---


Created an attachment (id=236779)
 View: https://bugzilla.gnome.org/attachment.cgi?id=236779
 Review: https://bugzilla.gnome.org/review?bug=694184&attachment=236779

Patch

In rtpsession.c, change_ssrc field of RTPSession should be set before calling 
rtp_session_schedule_bye_locked() as this function will call reconsider
function that will wake up rtcp_thread which will call rtp_session_on_timeout()
that will check change_ssrc to change the ssrc.
Indeed currently, there is a risk that rtp_session_on_timeout() is called
before change_ssrc is updated.

-- 
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.