[Bug 701587] rtsp-client: send new-session signal at the right time.

[GStreamer (bugzilla.gnome.org)]

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

--- Comment #2 from Wim Taymans <wim.taymans at gmail.com> 2013-06-17 15:32:11 UTC ---
@@ -1252,6 +1277,18 @@ handle_setup_request (GstRTSPClient * client,
GstRTSPClientState * state)

     /* we need a new media configuration in this session */
     sessmedia = NULL;
+
+    /* find the session in the session pool */
+    sessid = gst_rtsp_session_get_sessionid (session);
+    if (!(session = gst_rtsp_session_pool_find (priv->session_pool, sessid)))
+      goto no_session;

you get the sessionid of a session and then get the session with that id again?
also, the previous session would be leaked then.

 static void
 handle_request (GstRTSPClient * client, GstRTSPMessage * request)
@@ -1672,10 +1693,6 @@ handle_request (GstRTSPClient * client, GstRTSPMessage *
request)
     if (!(session = gst_rtsp_session_pool_find (priv->session_pool, sessid)))
       goto session_not_found;

-    /* we add the session to the client list of watched sessions. When a
session
-     * disappears because it times out, we will be notified. If all sessions
are
-     * gone, we will close the connection */
-    client_watch_session (client, session);
   }

This bothers me, if a new client connects with an old existing session, it will
not be tracked by this client, that's not right.

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