]> git.sesse.net Git - vlc/blobdiff - modules/stream_out/rtsp.c
Fix memleaks (corner case).
[vlc] / modules / stream_out / rtsp.c
index 5d252581b8351fb34a5fb2b70818b99d64cddba0..30e35526bc87da2150294ab9287a93cafb915265 100644 (file)
@@ -1,8 +1,10 @@
 /*****************************************************************************
  * rtsp.c: RTSP support for RTP stream output module
  *****************************************************************************
- * Copyright (C) 2003-2007 the VideoLAN team
- * $Id: rtp.c 21407 2007-08-22 20:10:41Z courmisch $
+ * Copyright (C) 2003-2004 the VideoLAN team
+ * Copyright © 2007 Rémi Denis-Courmont
+ *
+ * $Id$
  *
  * Authors: Laurent Aimar <fenrir@via.ecp.fr>
  *
 /*****************************************************************************
  * Preamble
  *****************************************************************************/
-#include <vlc/vlc.h>
+#ifdef HAVE_CONFIG_H
+# include "config.h"
+#endif
+
+#include <vlc_common.h>
 #include <vlc_sout.h>
 
 #include <vlc_httpd.h>
 #include <vlc_url.h>
 #include <vlc_network.h>
+#include <vlc_rand.h>
 #include <assert.h>
+#include <errno.h>
+#include <stdlib.h>
 
 #include "rtp.h"
 
@@ -42,8 +51,9 @@ struct rtsp_stream_t
     sout_stream_t  *owner;
     httpd_host_t   *host;
     httpd_url_t    *url;
-    char           *psz_control;
     char           *psz_path;
+    const char     *track_fmt;
+    unsigned        port;
 
     int             sessionc;
     rtsp_session_t **sessionv;
@@ -51,47 +61,52 @@ struct rtsp_stream_t
 
 
 static int  RtspCallback( httpd_callback_sys_t *p_args,
-                          httpd_client_t *cl,
-                          httpd_message_t *answer, httpd_message_t *query );
+                          httpd_client_t *cl, httpd_message_t *answer,
+                          const httpd_message_t *query );
 static int  RtspCallbackId( httpd_callback_sys_t *p_args,
-                            httpd_client_t *cl,
-                            httpd_message_t *answer, httpd_message_t *query );
+                            httpd_client_t *cl, httpd_message_t *answer,
+                            const httpd_message_t *query );
 static void RtspClientDel( rtsp_stream_t *rtsp, rtsp_session_t *session );
 
 rtsp_stream_t *RtspSetup( sout_stream_t *p_stream, const vlc_url_t *url )
 {
     rtsp_stream_t *rtsp = malloc( sizeof( *rtsp ) );
 
-    if( rtsp == NULL )
+    if( rtsp == NULL || ( url->i_port > 99999 ) )
+    {
+        free( rtsp );
         return NULL;
+    }
 
     rtsp->owner = p_stream;
     rtsp->sessionc = 0;
     rtsp->sessionv = NULL;
-    vlc_mutex_init( p_stream, &rtsp->lock );
+    rtsp->host = NULL;
+    rtsp->url = NULL;
+    rtsp->psz_path = NULL;
+    vlc_mutex_init( &rtsp->lock );
 
-    msg_Dbg( p_stream, "rtsp setup: %s : %d / %s\n",
-             url->psz_host, url->i_port, url->psz_path );
-
-    rtsp->psz_path = strdup( url->psz_path ? url->psz_path : "/" );
+    rtsp->port = (url->i_port > 0) ? url->i_port : 554;
+    rtsp->psz_path = strdup( ( url->psz_path != NULL ) ? url->psz_path : "/" );
     if( rtsp->psz_path == NULL )
         goto error;
 
-    if( asprintf( &rtsp->psz_control, "rtsp://%s:%d%s",
-                  url->psz_host,  url->i_port > 0 ? url->i_port : 554,
-                  rtsp->psz_path ) == -1 )
-    {
-        rtsp->psz_control = NULL;
-        goto error;
-    }
+    assert( strlen( rtsp->psz_path ) > 0 );
+    if( rtsp->psz_path[strlen( rtsp->psz_path ) - 1] == '/' )
+        rtsp->track_fmt = "%strackID=%u";
+    else
+        rtsp->track_fmt = "%s/trackID=%u";
+
+    msg_Dbg( p_stream, "RTSP stream: host %s port %d at %s",
+             url->psz_host, rtsp->port, rtsp->psz_path );
 
     rtsp->host = httpd_HostNew( VLC_OBJECT(p_stream), url->psz_host,
-                                url->i_port > 0 ? url->i_port : 554 );
+                                rtsp->port );
     if( rtsp->host == NULL )
         goto error;
 
-    rtsp->url = httpd_UrlNewUnique( rtsp->host, rtsp->psz_path, NULL, NULL,
-                                    NULL );
+    rtsp->url = httpd_UrlNewUnique( rtsp->host, rtsp->psz_path,
+                                    NULL, NULL, NULL );
     if( rtsp->url == NULL )
         goto error;
 
@@ -99,6 +114,8 @@ rtsp_stream_t *RtspSetup( sout_stream_t *p_stream, const vlc_url_t *url )
     httpd_UrlCatch( rtsp->url, HTTPD_MSG_SETUP,    RtspCallback, (void*)rtsp );
     httpd_UrlCatch( rtsp->url, HTTPD_MSG_PLAY,     RtspCallback, (void*)rtsp );
     httpd_UrlCatch( rtsp->url, HTTPD_MSG_PAUSE,    RtspCallback, (void*)rtsp );
+    httpd_UrlCatch( rtsp->url, HTTPD_MSG_GETPARAMETER, RtspCallback,
+                    (void*)rtsp );
     httpd_UrlCatch( rtsp->url, HTTPD_MSG_TEARDOWN, RtspCallback, (void*)rtsp );
     return rtsp;
 
@@ -119,7 +136,10 @@ void RtspUnsetup( rtsp_stream_t *rtsp )
     if( rtsp->host )
         httpd_HostDelete( rtsp->host );
 
+    free( rtsp->psz_path );
     vlc_mutex_destroy( &rtsp->lock );
+
+    free( rtsp );
 }
 
 
@@ -130,35 +150,41 @@ struct rtsp_stream_id_t
     httpd_url_t      *url;
     const char       *dst;
     int               ttl;
-    unsigned          loport, hiport;
+    uint32_t          ssrc;
+    uint16_t          loport, hiport;
 };
 
 
+typedef struct rtsp_strack_t rtsp_strack_t;
+
 /* For unicast streaming */
 struct rtsp_session_t
 {
     rtsp_stream_t *stream;
-
-    /* is it in "play" state */
-    vlc_bool_t     b_playing;
+    uint64_t       id;
 
     /* output (id-access) */
-    int               i_id;
-    sout_stream_id_t  **id;
-    int               i_access;
-    sout_access_out_t **access;
+    int            trackc;
+    rtsp_strack_t *trackv;
+};
 
-    char name[0];
+
+/* Unicast session track */
+struct rtsp_strack_t
+{
+    sout_stream_id_t  *id;
+    int                fd;
+    bool         playing;
 };
 
 
 rtsp_stream_id_t *RtspAddId( rtsp_stream_t *rtsp, sout_stream_id_t *sid,
-                             unsigned num,
+                             unsigned num, uint32_t ssrc,
                              /* Multicast stuff - TODO: cleanup */
                              const char *dst, int ttl,
                              unsigned loport, unsigned hiport )
 {
-    char urlbuf[strlen( rtsp->psz_control ) + 1 + 10];
+    char urlbuf[sizeof( "/trackID=123" ) + strlen( rtsp->psz_path )];
     rtsp_stream_id_t *id = malloc( sizeof( *id ) );
     httpd_url_t *url;
 
@@ -167,6 +193,7 @@ rtsp_stream_id_t *RtspAddId( rtsp_stream_t *rtsp, sout_stream_id_t *sid,
 
     id->stream = rtsp;
     id->sout_id = sid;
+    id->ssrc = ssrc;
     /* TODO: can we assume that this need not be strdup'd? */
     id->dst = dst;
     if( id->dst != NULL )
@@ -176,8 +203,10 @@ rtsp_stream_id_t *RtspAddId( rtsp_stream_t *rtsp, sout_stream_id_t *sid,
         id->hiport = hiport;
     }
 
-    sprintf( urlbuf, "%s/trackID=%d", rtsp->psz_path, num );
-    msg_Dbg( rtsp->owner, "RTSP: adding %s\n", urlbuf );
+    /* FIXME: num screws up if any ES has been removed and re-added */
+    snprintf( urlbuf, sizeof( urlbuf ), rtsp->track_fmt, rtsp->psz_path,
+              num );
+    msg_Dbg( rtsp->owner, "RTSP: adding %s", urlbuf );
     url = id->url = httpd_UrlNewUnique( rtsp->host, urlbuf, NULL, NULL, NULL );
 
     if( url == NULL )
@@ -190,6 +219,7 @@ rtsp_stream_id_t *RtspAddId( rtsp_stream_t *rtsp, sout_stream_id_t *sid,
     httpd_UrlCatch( url, HTTPD_MSG_SETUP,    RtspCallbackId, (void *)id );
     httpd_UrlCatch( url, HTTPD_MSG_PLAY,     RtspCallbackId, (void *)id );
     httpd_UrlCatch( url, HTTPD_MSG_PAUSE,    RtspCallbackId, (void *)id );
+    httpd_UrlCatch( url, HTTPD_MSG_GETPARAMETER, RtspCallbackId, (void *)id );
     httpd_UrlCatch( url, HTTPD_MSG_TEARDOWN, RtspCallbackId, (void *)id );
 
     return id;
@@ -203,16 +233,13 @@ void RtspDelId( rtsp_stream_t *rtsp, rtsp_stream_id_t *id )
     {
         rtsp_session_t *ses = rtsp->sessionv[i];
 
-        for( int j = 0; j < ses->i_id; j++ )
+        for( int j = 0; j < ses->trackc; j++ )
         {
-            if( ses->id[j] == id->sout_id )
+            if( ses->trackv[j].id == id->sout_id )
             {
-                REMOVE_ELEM( ses->id, ses->i_id, j );
-
-                assert( ses->access[j] != NULL );
-                sout_AccessOutDelete( ses->access[j] );
-                REMOVE_ELEM( ses->access, ses->i_access, j );
-                /* FIXME: are we supposed to notify the client? */
+                rtsp_strack_t *tr = ses->trackv + j;
+                net_Close( tr->fd );
+                REMOVE_ELEM( ses->trackv, ses->trackc, j );
             }
         }
     }
@@ -225,16 +252,16 @@ void RtspDelId( rtsp_stream_t *rtsp, rtsp_stream_id_t *id )
 
 /** rtsp must be locked */
 static
-rtsp_session_t *RtspClientNew( rtsp_stream_t *rtsp, const char *name )
+rtsp_session_t *RtspClientNew( rtsp_stream_t *rtsp )
 {
-    rtsp_session_t *s = malloc( sizeof( *s ) + strlen( name ) + 1 );
+    rtsp_session_t *s = malloc( sizeof( *s ) );
+    if( s == NULL )
+        return NULL;
 
     s->stream = rtsp;
-    s->b_playing = VLC_FALSE;
-    s->i_id = s->i_access = 0;
-    s->id = NULL;
-    s->access = NULL;
-    strcpy( s->name, name );
+    vlc_rand_bytes (&s->id, sizeof (s->id));
+    s->trackc = 0;
+    s->trackv = NULL;
 
     TAB_APPEND( rtsp->sessionc, rtsp->sessionv, s );
 
@@ -246,15 +273,22 @@ rtsp_session_t *RtspClientNew( rtsp_stream_t *rtsp, const char *name )
 static
 rtsp_session_t *RtspClientGet( rtsp_stream_t *rtsp, const char *name )
 {
+    char *end;
+    uint64_t id;
     int i;
 
     if( name == NULL )
         return NULL;
 
+    errno = 0;
+    id = strtoull( name, &end, 0x10 );
+    if( errno || *end )
+        return NULL;
+
     /* FIXME: use a hash/dictionary */
     for( i = 0; i < rtsp->sessionc; i++ )
     {
-        if( !strcmp( rtsp->sessionv[i]->name, name ) )
+        if( rtsp->sessionv[i]->id == id )
             return rtsp->sessionv[i];
     }
     return NULL;
@@ -268,123 +302,14 @@ void RtspClientDel( rtsp_stream_t *rtsp, rtsp_session_t *session )
     int i;
     TAB_REMOVE( rtsp->sessionc, rtsp->sessionv, session );
 
-    for( i = 0; i < session->i_access; i++ )
-    {
-        rtp_del_sink( session->id[i], session->access[i] );
-        sout_AccessOutDelete( session->access[i] );
-    }
+    for( i = 0; i < session->trackc; i++ )
+        rtp_del_sink( session->trackv[i].id, session->trackv[i].fd );
 
-    free( session->id );
-    free( session->access );
+    free( session->trackv );
     free( session );
 }
 
 
-/** Aggregate RTSP callback */
-static int  RtspCallback( httpd_callback_sys_t *p_args,
-                          httpd_client_t *cl,
-                          httpd_message_t *answer, httpd_message_t *query )
-{
-    rtsp_stream_t *rtsp = (rtsp_stream_t *)p_args;
-    const char *psz_session = NULL, *psz_cseq;
-
-    if( answer == NULL || query == NULL )
-    {
-        return VLC_SUCCESS;
-    }
-    //fprintf( stderr, "RtspCallback query: type=%d\n", query->i_type );
-
-    answer->i_proto = HTTPD_PROTO_RTSP;
-    answer->i_version= query->i_version;
-    answer->i_type   = HTTPD_MSG_ANSWER;
-    answer->i_body = 0;
-    answer->p_body = NULL;
-
-    if( httpd_MsgGet( query, "Require" ) != NULL )
-    {
-        answer->i_status = 551;
-        httpd_MsgAdd( query, "Unsupported", "%s",
-                      httpd_MsgGet( query, "Require" ) );
-    }
-    else
-    switch( query->i_type )
-    {
-        case HTTPD_MSG_DESCRIBE:
-        {
-            char *psz_sdp = SDPGenerate( rtsp->owner, rtsp->psz_control );
-
-            answer->i_status = 200;
-            httpd_MsgAdd( answer, "Content-Type",  "%s", "application/sdp" );
-            httpd_MsgAdd( answer, "Content-Base",  "%s", rtsp->psz_control );
-            answer->p_body = (uint8_t *)psz_sdp;
-            answer->i_body = strlen( psz_sdp );
-            break;
-        }
-
-        case HTTPD_MSG_SETUP:
-            answer->i_status = 459;
-            break;
-
-        case HTTPD_MSG_PLAY:
-        {
-            rtsp_session_t *ses;
-            answer->i_status = 200;
-
-            psz_session = httpd_MsgGet( query, "Session" );
-
-            vlc_mutex_lock( &rtsp->lock );
-            ses = RtspClientGet( rtsp, psz_session );
-            if( ( ses != NULL ) && !ses->b_playing )
-            {
-                /* FIXME */
-                ses->b_playing = VLC_TRUE;
-
-                for( int i_id = 0; i_id < ses->i_id; i_id++ )
-                    rtp_add_sink( ses->id[i_id], ses->access[i_id] );
-            }
-            vlc_mutex_unlock( &rtsp->lock );
-            break;
-        }
-
-        case HTTPD_MSG_PAUSE:
-            answer->i_status = 405;
-            httpd_MsgAdd( answer, "Allow", "DESCRIBE, PLAY, TEARDOWN" );
-            break;
-
-        case HTTPD_MSG_TEARDOWN:
-        {
-            rtsp_session_t *ses;
-
-            /* for now only multicast so easy again */
-            answer->i_status = 200;
-
-            psz_session = httpd_MsgGet( query, "Session" );
-
-            vlc_mutex_lock( &rtsp->lock );
-            ses = RtspClientGet( rtsp, psz_session );
-            if( ses != NULL )
-                RtspClientDel( rtsp, ses );
-            vlc_mutex_unlock( &rtsp->lock );
-            break;
-        }
-
-        default:
-            return VLC_EGENERIC;
-    }
-
-    httpd_MsgAdd( answer, "Server", "%s", PACKAGE_STRING );
-    httpd_MsgAdd( answer, "Content-Length", "%d", answer->i_body );
-    psz_cseq = httpd_MsgGet( query, "Cseq" );
-    if( psz_cseq )
-        httpd_MsgAdd( answer, "Cseq", "%s", psz_cseq );
-    httpd_MsgAdd( answer, "Cache-Control", "%s", "no-cache" );
-
-    if( psz_session )
-        httpd_MsgAdd( answer, "Session", "%s;timeout=5", psz_session );
-    return VLC_SUCCESS;
-}
-
-
 /** Finds the next transport choice */
 static inline const char *transport_next( const char *str )
 {
@@ -411,56 +336,116 @@ static inline const char *parameter_next( const char *str )
 }
 
 
-/** Non-aggregate RTSP callback */
-static int RtspCallbackId( httpd_callback_sys_t *p_args,
-                           httpd_client_t *cl,
-                           httpd_message_t *answer, httpd_message_t *query )
+/** RTSP requests handler
+ * @param id selected track for non-aggregate URLs,
+ *           NULL for aggregate URLs
+ */
+static int RtspHandler( rtsp_stream_t *rtsp, rtsp_stream_id_t *id,
+                        httpd_client_t *cl,
+                        httpd_message_t *answer,
+                        const httpd_message_t *query )
 {
-    rtsp_stream_id_t *id = (rtsp_stream_id_t *)p_args;
-    sout_stream_t    *p_stream = id->stream->owner;
-    sout_stream_id_t *sid = id->sout_id;
-    char psz_session_init[21];
-    const char *psz_session;
-    const char *psz_cseq;
+    sout_stream_t *p_stream = rtsp->owner;
+    char psz_sesbuf[17];
+    const char *psz_session = NULL, *psz;
+    char control[sizeof("rtsp://[]:12345") + NI_MAXNUMERICHOST
+                  + strlen( rtsp->psz_path )];
+    time_t now;
 
-    if( answer == NULL || query == NULL )
+    time (&now);
+
+    if( answer == NULL || query == NULL || cl == NULL )
         return VLC_SUCCESS;
-    //fprintf( stderr, "RtspCallback query: type=%d\n", query->i_type );
+    else
+    {
+        /* Build self-referential control URL */
+        char ip[NI_MAXNUMERICHOST], *ptr;
+
+        httpd_ServerIP( cl, ip );
+        ptr = strchr( ip, '%' );
+        if( ptr != NULL )
+            *ptr = '\0';
+
+        if( strchr( ip, ':' ) != NULL )
+            sprintf( control, "rtsp://[%s]:%u%s", ip, rtsp->port,
+                     rtsp->psz_path );
+        else
+            sprintf( control, "rtsp://%s:%u%s", ip, rtsp->port,
+                     rtsp->psz_path );
+    }
 
     /* */
     answer->i_proto = HTTPD_PROTO_RTSP;
-    answer->i_version= query->i_version;
+    answer->i_version= 0;
     answer->i_type   = HTTPD_MSG_ANSWER;
     answer->i_body = 0;
     answer->p_body = NULL;
 
-    /* Create new session ID if needed */
-    psz_session = httpd_MsgGet( query, "Session" );
-    if( psz_session == NULL )
-    {
-        /* FIXME: should be somewhat secure randomness */
-        snprintf( psz_session_init, sizeof(psz_session_init), I64Fd,
-                  NTPtime64() + rand() );
+    httpd_MsgAdd( answer, "Server", "%s", PACKAGE_STRING );
+
+    /* Date: is always allowed, and sometimes mandatory with RTSP/2.0. */
+    struct tm ut;
+    if (gmtime_r (&now, &ut) != NULL)
+    {   /* RFC1123 format, GMT is mandatory */
+        static const char wdays[7][4] = {
+            "Sun", "Mon", "Tue", "Wed", "Thu", "Fri", "Sat" };
+        static const char mons[12][4] = {
+            "Jan", "Feb", "Mar", "Apr", "May", "Jun",
+            "Jul", "Aug", "Sep", "Oct", "Nov", "Dec" };
+        httpd_MsgAdd (answer, "Date", "%s, %02u %s %04u %02u:%02u:%02u GMT",
+                      wdays[ut.tm_wday], ut.tm_mday, mons[ut.tm_mon],
+                      1900 + ut.tm_year, ut.tm_hour, ut.tm_min, ut.tm_sec);
     }
 
+    if( query->i_proto != HTTPD_PROTO_RTSP )
+    {
+        answer->i_status = 505;
+    }
+    else
     if( httpd_MsgGet( query, "Require" ) != NULL )
     {
         answer->i_status = 551;
-        httpd_MsgAdd( query, "Unsupported", "%s",
+        httpd_MsgAdd( answer, "Unsupported", "%s",
                       httpd_MsgGet( query, "Require" ) );
     }
     else
     switch( query->i_type )
     {
+        case HTTPD_MSG_DESCRIBE:
+        {   /* Aggregate-only */
+            if( id != NULL )
+            {
+                answer->i_status = 460;
+                break;
+            }
+
+            answer->i_status = 200;
+            httpd_MsgAdd( answer, "Content-Type",  "%s", "application/sdp" );
+            httpd_MsgAdd( answer, "Content-Base",  "%s", control );
+            answer->p_body = (uint8_t *)SDPGenerate( rtsp->owner, control );
+            if( answer->p_body != NULL )
+                answer->i_body = strlen( (char *)answer->p_body );
+            else
+                answer->i_status = 500;
+            break;
+        }
+
         case HTTPD_MSG_SETUP:
-        {
+            /* Non-aggregate-only */
+            if( id == NULL )
+            {
+                answer->i_status = 459;
+                break;
+            }
+
+            psz_session = httpd_MsgGet( query, "Session" );
             answer->i_status = 461;
 
             for( const char *tpt = httpd_MsgGet( query, "Transport" );
                  tpt != NULL;
                  tpt = transport_next( tpt ) )
             {
-                vlc_bool_t b_multicast = VLC_TRUE, b_unsupp = VLC_FALSE;
+                bool b_multicast = true, b_unsupp = false;
                 unsigned loport = 5004, hiport = 5005; /* from RFC3551 */
 
                 /* Check transport protocol. */
@@ -479,12 +464,13 @@ static int RtspCallbackId( httpd_callback_sys_t *p_args,
                      opt = parameter_next( opt ) )
                 {
                     if( strncmp( opt, "multicast", 9 ) == 0)
-                        b_multicast = VLC_TRUE;
+                        b_multicast = true;
                     else
                     if( strncmp( opt, "unicast", 7 ) == 0 )
-                        b_multicast = VLC_FALSE;
+                        b_multicast = false;
                     else
-                    if( sscanf( opt, "client_port=%u-%u", &loport, &hiport ) == 2 )
+                    if( sscanf( opt, "client_port=%u-%u", &loport, &hiport )
+                                == 2 )
                         ;
                     else
                     if( strncmp( opt, "mode=", 5 ) == 0 )
@@ -493,26 +479,32 @@ static int RtspCallbackId( httpd_callback_sys_t *p_args,
                          && strncasecmp( opt + 5, "\"PLAY\"", 6 ) )
                         {
                             /* Not playing?! */
-                            b_unsupp = VLC_TRUE;
+                            b_unsupp = true;
                             break;
                         }
                     }
                     else
+                    if( strncmp( opt,"destination=", 12 ) == 0 )
+                    {
+                        answer->i_status = 403;
+                        b_unsupp = true;
+                    }
+                    else
                     {
                     /*
                      * Every other option is unsupported:
                      *
-                     * "source" and "append" are invalid.
+                     * "source" and "append" are invalid (server-only);
+                     * "ssrc" also (as clarified per RFC2326bis).
                      *
                      * For multicast, "port", "layers", "ttl" are set by the
                      * stream output configuration.
                      *
-                     * For unicast, we do not allow "destination" as it
-                     * carries a DoS risk, and we decide on "server_port".
+                     * For unicast, we want to decide "server_port" values.
                      *
-                     * "interleaved" and "ssrc" are not implemented.
+                     * "interleaved" is not implemented.
                      */
-                        b_unsupp = VLC_TRUE;
+                        b_unsupp = true;
                         break;
                     }
                 }
@@ -526,6 +518,13 @@ static int RtspCallbackId( httpd_callback_sys_t *p_args,
                     if( dst == NULL )
                         continue;
 
+                    if( psz_session == NULL )
+                    {
+                        /* Create a dummy session ID */
+                        snprintf( psz_sesbuf, sizeof( psz_sesbuf ), "%d",
+                                  rand() );
+                        psz_session = psz_sesbuf;
+                    }
                     answer->i_status = 200;
 
                     httpd_MsgAdd( answer, "Transport",
@@ -536,10 +535,10 @@ static int RtspCallbackId( httpd_callback_sys_t *p_args,
                 }
                 else
                 {
-                    char ip[NI_MAXNUMERICHOST], url[NI_MAXNUMERICHOST + 8];
-                    static const char access[] = "udp{raw,rtcp}";
-                    sout_access_out_t *p_access;
+                    char ip[NI_MAXNUMERICHOST], src[NI_MAXNUMERICHOST];
                     rtsp_session_t *ses = NULL;
+                    rtsp_strack_t track = { id->sout_id, -1, false };
+                    int sport;
 
                     if( httpd_ClientIP( cl, ip ) == NULL )
                     {
@@ -547,52 +546,47 @@ static int RtspCallbackId( httpd_callback_sys_t *p_args,
                         continue;
                     }
 
-                    snprintf( url, sizeof( url ),
-                              ( strchr( ip, ':' ) != NULL ) ? "[%s]:%d" : "%s:%d",
-                              ip, loport );
-
-                    p_access = sout_AccessOutNew( p_stream->p_sout, access,
-                                                  url );
-                    if( p_access == NULL )
+                    track.fd = net_ConnectDgram( p_stream, ip, loport, -1,
+                                                 IPPROTO_UDP );
+                    if( track.fd == -1 )
                     {
                         msg_Err( p_stream,
-                                 "cannot create access output for %s://%s",
-                                 access, url );
+                                 "cannot create RTP socket for %s port %u",
+                                 ip, loport );
                         answer->i_status = 500;
                         continue;
                     }
 
-                    vlc_mutex_lock( &id->stream->lock );
+                    net_GetSockAddress( track.fd, src, &sport );
+
+                    vlc_mutex_lock( &rtsp->lock );
                     if( psz_session == NULL )
                     {
-                        psz_session = psz_session_init;
-                        ses = RtspClientNew( id->stream, psz_session );
+                        ses = RtspClientNew( rtsp );
+                        snprintf( psz_sesbuf, sizeof( psz_sesbuf ), "%"PRIx64,
+                                  ses->id );
+                        psz_session = psz_sesbuf;
                     }
                     else
                     {
                         /* FIXME: we probably need to remove an access out,
                          * if there is already one for the same ID */
-                        ses = RtspClientGet( id->stream, psz_session );
+                        ses = RtspClientGet( rtsp, psz_session );
                         if( ses == NULL )
                         {
                             answer->i_status = 454;
-                            vlc_mutex_unlock( &id->stream->lock );
+                            vlc_mutex_unlock( &rtsp->lock );
                             continue;
                         }
                     }
 
-                    assert( ses->i_id == ses->i_access );
-                    TAB_APPEND( ses->i_id, ses->id, sid );
-                    TAB_APPEND( ses->i_access, ses->access, p_access );
-                    assert( ses->i_id == ses->i_access );
-                    vlc_mutex_unlock( &id->stream->lock );
-
-                    char *src = var_GetNonEmptyString (p_access, "src-addr");
-                    int sport = var_GetInteger (p_access, "src-port");
+                    INSERT_ELEM( ses->trackv, ses->trackc, ses->trackc,
+                                 track );
+                    vlc_mutex_unlock( &rtsp->lock );
 
                     httpd_ServerIP( cl, ip );
 
-                    if( ( src != NULL ) && strcmp( src, ip ) )
+                    if( strcmp( src, ip ) )
                     {
                         /* Specify source IP if it is different from the RTSP
                          * control connection server address */
@@ -602,39 +596,160 @@ static int RtspCallbackId( httpd_callback_sys_t *p_args,
                         httpd_MsgAdd( answer, "Transport",
                                       "RTP/AVP/UDP;unicast;source=%s;"
                                       "client_port=%u-%u;server_port=%u-%u;"
-                                      "mode=play",
-                                      src, loport, loport + 1, sport, sport + 1 );
+                                      "ssrc=%08X;mode=play",
+                                      src, loport, loport + 1, sport,
+                                      sport + 1, id->ssrc );
                     }
                     else
                     {
                         httpd_MsgAdd( answer, "Transport",
                                       "RTP/AVP/UDP;unicast;"
                                       "client_port=%u-%u;server_port=%u-%u;"
-                                      "mode=play",
-                                      loport, loport + 1, sport, sport + 1 );
+                                      "ssrc=%08X;mode=play",
+                                      loport, loport + 1, sport, sport + 1,
+                                      id->ssrc );
                     }
 
                     answer->i_status = 200;
-                    free( src );
                 }
                 break;
             }
             break;
+
+        case HTTPD_MSG_PLAY:
+        {
+            rtsp_session_t *ses;
+            answer->i_status = 200;
+
+            psz_session = httpd_MsgGet( query, "Session" );
+            const char *range = httpd_MsgGet (query, "Range");
+            if (range && strncmp (range, "npt=", 4))
+            {
+                answer->i_status = 501;
+                break;
+            }
+
+            vlc_mutex_lock( &rtsp->lock );
+            ses = RtspClientGet( rtsp, psz_session );
+            if( ses != NULL )
+            {
+                /* FIXME: we really need to limit the number of tracks... */
+                char info[ses->trackc * ( strlen( control )
+                                  + sizeof("/trackID=123;seq=65535, ") ) + 1];
+                size_t infolen = 0;
+
+                for( int i = 0; i < ses->trackc; i++ )
+                {
+                    rtsp_strack_t *tr = ses->trackv + i;
+                    if( ( id == NULL ) || ( tr->id == id->sout_id ) )
+                    {
+                        if( !tr->playing )
+                        {
+                            tr->playing = true;
+                            rtp_add_sink( tr->id, tr->fd, false );
+                        }
+                        infolen += sprintf( info + infolen,
+                                            "%s/trackID=%u;seq=%u, ", control,
+                                            rtp_get_num( tr->id ),
+                                            rtp_get_seq( tr->id ) );
+                    }
+                }
+                if( infolen > 0 )
+                {
+                    info[infolen - 2] = '\0'; /* remove trailing ", " */
+                    httpd_MsgAdd( answer, "RTP-Info", "%s", info );
+                }
+            }
+            vlc_mutex_unlock( &rtsp->lock );
+
+            if( httpd_MsgGet( query, "Scale" ) != NULL )
+                httpd_MsgAdd( answer, "Scale", "1." );
+            break;
         }
 
-        default:
-            answer->i_status = 460;
+        case HTTPD_MSG_PAUSE:
+            answer->i_status = 405;
+            httpd_MsgAdd( answer, "Allow",
+                          "%s, TEARDOWN, PLAY, GET_PARAMETER",
+                          ( id != NULL ) ? "SETUP" : "DESCRIBE" );
             break;
-    }
 
-    psz_cseq = httpd_MsgGet( query, "Cseq" );
-    if( psz_cseq )
-        httpd_MsgAdd( answer, "Cseq", "%s", psz_cseq );
-    httpd_MsgAdd( answer, "Server", "%s", PACKAGE_STRING );
-    httpd_MsgAdd( answer, "Content-Length", "%d", answer->i_body );
-    httpd_MsgAdd( answer, "Cache-Control", "%s", "no-cache" );
+        case HTTPD_MSG_GETPARAMETER:
+            if( query->i_body > 0 )
+            {
+                answer->i_status = 451;
+                break;
+            }
+
+            psz_session = httpd_MsgGet( query, "Session" );
+            answer->i_status = 200;
+            break;
+
+        case HTTPD_MSG_TEARDOWN:
+        {
+            rtsp_session_t *ses;
+
+            answer->i_status = 200;
+
+            psz_session = httpd_MsgGet( query, "Session" );
+
+            vlc_mutex_lock( &rtsp->lock );
+            ses = RtspClientGet( rtsp, psz_session );
+            if( ses != NULL )
+            {
+                if( id == NULL ) /* Delete the entire session */
+                    RtspClientDel( rtsp, ses );
+                else /* Delete one track from the session */
+                for( int i = 0; i < ses->trackc; i++ )
+                {
+                    if( ses->trackv[i].id == id->sout_id )
+                    {
+                        rtp_del_sink( id->sout_id, ses->trackv[i].fd );
+                        REMOVE_ELEM( ses->trackv, ses->trackc, i );
+                    }
+                }
+            }
+            vlc_mutex_unlock( &rtsp->lock );
+            break;
+        }
+
+        default:
+            return VLC_EGENERIC;
+    }
 
     if( psz_session )
         httpd_MsgAdd( answer, "Session", "%s"/*;timeout=5*/, psz_session );
+
+    httpd_MsgAdd( answer, "Content-Length", "%d", answer->i_body );
+    httpd_MsgAdd( answer, "Cache-Control", "no-cache" );
+
+    psz = httpd_MsgGet( query, "Cseq" );
+    if( psz != NULL )
+        httpd_MsgAdd( answer, "Cseq", "%s", psz );
+    psz = httpd_MsgGet( query, "Timestamp" );
+    if( psz != NULL )
+        httpd_MsgAdd( answer, "Timestamp", "%s", psz );
+
     return VLC_SUCCESS;
 }
+
+
+/** Aggregate RTSP callback */
+static int RtspCallback( httpd_callback_sys_t *p_args,
+                         httpd_client_t *cl,
+                         httpd_message_t *answer,
+                         const httpd_message_t *query )
+{
+    return RtspHandler( (rtsp_stream_t *)p_args, NULL, cl, answer, query );
+}
+
+
+/** Non-aggregate RTSP callback */
+static int RtspCallbackId( httpd_callback_sys_t *p_args,
+                           httpd_client_t *cl,
+                           httpd_message_t *answer,
+                           const httpd_message_t *query )
+{
+    rtsp_stream_id_t *id = (rtsp_stream_id_t *)p_args;
+    return RtspHandler( id->stream, id, cl, answer, query );
+}