[Bug 732226] Races when sessions time out

[GStreamer (bugzilla.gnome.org)]

https://bugzilla.gnome.org/show_bug.cgi?id=732226
  GStreamer | gst-rtsp-server | git

Ognyan Tonchev <ognyan.tonchev> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
 Attachment #279215|0                           |1
        is obsolete|                            |

--- Comment #2 from Ognyan Tonchev <ognyan.tonchev at axis.com> 2014-06-25 17:43:54 UTC ---
Created an attachment (id=279239)
 View: https://bugzilla.gnome.org/attachment.cgi?id=279239
 Review: https://bugzilla.gnome.org/review?bug=732226&attachment=279239

Keep hard refs to the sessions from the client object

Slightly more correct version of the patch.

I should also mention that due to the extra client ref owned by the session
expired handler, the RTSPClient object will only be freed after either the
client sends TEARDOWN or the session expires. In both cases the expired handler
will be removed and the extra client ref will be dropped.

But if there is no session cleanup mechanism enabled for the server and if a
client never sends TEARDOWN then the RTSPClient object will be leaked. Which is
not very good...

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