X-Git-Url: https://git.sesse.net/?a=blobdiff_plain;f=modules%2Fstream_out%2Frtp.c;h=b5ef13d3a603b08ef09f09cd1e06282c072b9301;hb=9f5bb41a34b73cca36ad83c1fd563fe2b2b7fd4a;hp=124d685336ade1ec914ca1c50068932079942a00;hpb=2039a06bce2fa27120f596b9c9760f650a79f782;p=vlc diff --git a/modules/stream_out/rtp.c b/modules/stream_out/rtp.c index 124d685336..b5ef13d3a6 100644 --- a/modules/stream_out/rtp.c +++ b/modules/stream_out/rtp.c @@ -37,7 +37,7 @@ #include #include #include -#include +#include #include #include #ifdef HAVE_SRTP @@ -49,8 +49,6 @@ #ifdef HAVE_UNISTD_H # include # include -# include -# include #endif #ifdef HAVE_ARPA_INET_H # include @@ -246,7 +244,7 @@ static int MuxSend( sout_stream_t *, sout_stream_id_t *, block_t* ); static sout_access_out_t *GrabberCreate( sout_stream_t *p_sout ); -static void* ThreadSend( vlc_object_t *p_this ); +static void* ThreadSend( void * ); static void *rtp_listen_thread( void * ); static void SDPHandleUrl( sout_stream_t *, const char * ); @@ -313,8 +311,6 @@ typedef struct rtp_sink_t struct sout_stream_id_t { - VLC_COMMON_MEMBERS - sout_stream_t *p_stream; /* rtp field */ uint16_t i_sequence; @@ -343,6 +339,7 @@ struct sout_stream_id_t pf_rtp_packetizer_t pf_packetize; /* Packets sinks */ + vlc_thread_t thread; vlc_mutex_t lock_sink; int sinkc; rtp_sink_t *sinkv; @@ -459,10 +456,10 @@ static int Open( vlc_object_t *p_this ) if( p_sys->i_ttl == -1 ) { /* Normally, we should let the default hop limit up to the core, - * but we have to know it to build our SDP properly, which is why - * we ask the core. FIXME: broken when neither sout-rtp-ttl nor - * ttl are set. */ - p_sys->i_ttl = config_GetInt( p_stream, "ttl" ); + * but we have to know it to write our RTSP headers properly, + * which is why we ask the core. FIXME: broken when neither + * sout-rtp-ttl nor ttl are set. */ + p_sys->i_ttl = var_InheritInteger( p_stream, "ttl" ); } p_sys->b_latm = var_GetBool( p_stream, SOUT_CFG_PREFIX "mp4a-latm" ); @@ -475,7 +472,7 @@ static int Open( vlc_object_t *p_this ) p_sys->i_npt_zero = VLC_TS_INVALID; p_sys->i_pts_zero = mdate(); /* arbitrary value, could probably be * random */ - p_sys->payload_bitmap = 0; + p_sys->payload_bitmap = 0xFFFFFFFF; p_sys->i_es = 0; p_sys->es = NULL; p_sys->rtsp = NULL; @@ -683,7 +680,7 @@ static void SDPHandleUrl( sout_stream_t *p_stream, const char *psz_url ) if( p_sys->p_mux != NULL ) { sout_stream_id_t *id = p_sys->es[0]; - id->rtsp_id = RtspAddId( p_sys->rtsp, id, 0, GetDWBE( id->ssrc ), + id->rtsp_id = RtspAddId( p_sys->rtsp, id, GetDWBE( id->ssrc ), p_sys->psz_destination, p_sys->i_ttl, id->i_port, id->i_port + 1 ); } @@ -701,13 +698,9 @@ static void SDPHandleUrl( sout_stream_t *p_stream, const char *psz_url ) msg_Err( p_stream, "you can use sdp=file:// only once" ); goto out; } - psz_url = &psz_url[5]; - if( psz_url[0] == '/' && psz_url[1] == '/' ) - psz_url += 2; - p_sys->psz_sdp_file = strdup( psz_url ); + p_sys->psz_sdp_file = make_path( psz_url ); if( p_sys->psz_sdp_file == NULL ) goto out; - decode_URI( p_sys->psz_sdp_file ); /* FIXME? */ FileSetup( p_stream ); } else @@ -724,10 +717,10 @@ out: * SDPGenerate *****************************************************************************/ /*static*/ -char *SDPGenerate( const sout_stream_t *p_stream, const char *rtsp_url ) +char *SDPGenerate( sout_stream_t *p_stream, const char *rtsp_url ) { - const sout_stream_sys_t *p_sys = p_stream->p_sys; - char *psz_sdp; + sout_stream_sys_t *p_sys = p_stream->p_sys; + char *psz_sdp = NULL; struct sockaddr_storage dst; socklen_t dstlen; int i; @@ -748,11 +741,15 @@ char *SDPGenerate( const sout_stream_t *p_stream, const char *rtsp_url ) */ int inclport; + vlc_mutex_lock( &p_sys->lock_es ); + if( unlikely(p_sys->i_es == 0) ) + goto out; /* hmm... */ + if( p_sys->psz_destination != NULL ) { inclport = 1; - /* Oh boy, this is really ugly! (+ race condition on lock_es) */ + /* Oh boy, this is really ugly! */ dstlen = sizeof( dst ); if( p_sys->es[0]->listen.fd != NULL ) getsockname( p_sys->es[0]->listen.fd[0], @@ -765,19 +762,24 @@ char *SDPGenerate( const sout_stream_t *p_stream, const char *rtsp_url ) { inclport = 0; + /* Check against URL format rtsp://[]:/ */ + bool ipv6 = rtsp_url != NULL && strlen( rtsp_url ) > 7 + && rtsp_url[7] == '['; + /* Dummy destination address for RTSP */ - memset (&dst, 0, sizeof( struct sockaddr_in ) ); - dst.ss_family = AF_INET; + dstlen = ipv6 ? sizeof( struct sockaddr_in6 ) + : sizeof( struct sockaddr_in ); + memset (&dst, 0, dstlen); + dst.ss_family = ipv6 ? AF_INET6 : AF_INET; #ifdef HAVE_SA_LEN - dst.ss_len = + dst.ss_len = dstlen; #endif - dstlen = sizeof( struct sockaddr_in ); } psz_sdp = vlc_sdp_Start( VLC_OBJECT( p_stream ), SOUT_CFG_PREFIX, NULL, 0, (struct sockaddr *)&dst, dstlen ); if( psz_sdp == NULL ) - return NULL; + goto out; /* TODO: a=source-filter */ if( p_sys->rtcp_mux ) @@ -786,12 +788,28 @@ char *SDPGenerate( const sout_stream_t *p_stream, const char *rtsp_url ) if( rtsp_url != NULL ) sdp_AddAttribute ( &psz_sdp, "control", "%s", rtsp_url ); - /* FIXME: locking?! */ + const char *proto = "RTP/AVP"; /* protocol */ + if( rtsp_url == NULL ) + { + switch( p_sys->proto ) + { + case IPPROTO_UDP: + break; + case IPPROTO_TCP: + proto = "TCP/RTP/AVP"; + break; + case IPPROTO_DCCP: + proto = "DCCP/RTP/AVP"; + break; + case IPPROTO_UDPLITE: + return psz_sdp; + } + } + for( i = 0; i < p_sys->i_es; i++ ) { sout_stream_id_t *id = p_sys->es[i]; const char *mime_major; /* major MIME type */ - const char *proto = "RTP/AVP"; /* protocol */ switch( id->i_cat ) { @@ -808,38 +826,23 @@ char *SDPGenerate( const sout_stream_t *p_stream, const char *rtsp_url ) continue; } - if( rtsp_url == NULL ) - { - switch( p_sys->proto ) - { - case IPPROTO_UDP: - break; - case IPPROTO_TCP: - proto = "TCP/RTP/AVP"; - break; - case IPPROTO_DCCP: - proto = "DCCP/RTP/AVP"; - break; - case IPPROTO_UDPLITE: - continue; - } - } - sdp_AddMedia( &psz_sdp, mime_major, proto, inclport * id->i_port, id->i_payload_type, false, id->i_bitrate, id->psz_enc, id->i_clock_rate, id->i_channels, id->psz_fmtp); - if( !p_sys->rtcp_mux && (id->i_port & 1) ) /* cf RFC4566 §5.14 */ + /* cf RFC4566 §5.14 */ + if( inclport && !p_sys->rtcp_mux && (id->i_port & 1) ) sdp_AddAttribute ( &psz_sdp, "rtcp", "%u", id->i_port + 1 ); if( rtsp_url != NULL ) { - assert( strlen( rtsp_url ) > 0 ); - bool addslash = ( rtsp_url[strlen( rtsp_url ) - 1] != '/' ); - sdp_AddAttribute ( &psz_sdp, "control", - addslash ? "%s/trackID=%u" : "%strackID=%u", - rtsp_url, i ); + char *track_url = RtspAppendTrackPath( id->rtsp_id, rtsp_url ); + if( track_url != NULL ) + { + sdp_AddAttribute ( &psz_sdp, "control", "%s", track_url ); + free( track_url ); + } } else { @@ -850,7 +853,8 @@ char *SDPGenerate( const sout_stream_t *p_stream, const char *rtsp_url ) "SC:RTP%c", toupper( mime_major[0] ) ); } } - +out: + vlc_mutex_unlock( &p_sys->lock_es ); return psz_sdp; } @@ -861,9 +865,8 @@ char *SDPGenerate( const sout_stream_t *p_stream, const char *rtsp_url ) static void sprintf_hexa( char *s, uint8_t *p_data, int i_data ) { static const char hex[16] = "0123456789abcdef"; - int i; - for( i = 0; i < i_data; i++ ) + for( int i = 0; i < i_data; i++ ) { s[2*i+0] = hex[(p_data[i]>>4)&0xf]; s[2*i+1] = hex[(p_data[i] )&0xf]; @@ -901,68 +904,30 @@ static sout_stream_id_t *Add( sout_stream_t *p_stream, es_format_t *p_fmt ) /* NOTE: As a special case, if we use a non-RTP * mux (TS/PS), then p_fmt is NULL. */ sout_stream_sys_t *p_sys = p_stream->p_sys; - sout_stream_id_t *id; char *psz_sdp; - if (0xffffffff == p_sys->payload_bitmap) + if (0 == p_sys->payload_bitmap) { msg_Err (p_stream, "too many RTP elementary streams"); return NULL; } - /* Choose the port */ - uint16_t i_port = 0; - if( p_fmt == NULL ) - ; - else - if( p_fmt->i_cat == AUDIO_ES && p_sys->i_port_audio > 0 ) - i_port = p_sys->i_port_audio; - else - if( p_fmt->i_cat == VIDEO_ES && p_sys->i_port_video > 0 ) - i_port = p_sys->i_port_video; - - /* We do not need the ES lock (p_sys->lock_es) here, because this is the - * only one thread that can *modify* the ES table. The ES lock protects - * the other threads from our modifications (TAB_APPEND, TAB_REMOVE). */ - for (int i = 0; i_port && (i < p_sys->i_es); i++) - if (i_port == p_sys->es[i]->i_port) - i_port = 0; /* Port already in use! */ - for (uint16_t p = p_sys->i_port; i_port == 0; p += 2) - { - if (p == 0) - { - msg_Err (p_stream, "too many RTP elementary streams"); - return NULL; - } - i_port = p; - for (int i = 0; i_port && (i < p_sys->i_es); i++) - if (p == p_sys->es[i]->i_port) - i_port = 0; - } - - id = vlc_object_create( p_stream, sizeof( sout_stream_id_t ) ); - if( id == NULL ) + sout_stream_id_t *id = malloc( sizeof( *id ) ); + if( unlikely(id == NULL) ) return NULL; - vlc_object_attach( id, p_stream ); - id->p_stream = p_stream; /* Look for free dymanic payload type */ - id->i_payload_type = 96; - while (p_sys->payload_bitmap & (1 << (id->i_payload_type - 96))) - id->i_payload_type++; + id->i_payload_type = 96 + clz32 (p_sys->payload_bitmap); assert (id->i_payload_type < 128); vlc_rand_bytes (&id->i_sequence, sizeof (id->i_sequence)); vlc_rand_bytes (id->ssrc, sizeof (id->ssrc)); - id->i_seq_sent_next = id->i_sequence; - id->psz_enc = NULL; id->psz_fmtp = NULL; id->i_clock_rate = 90000; /* most common case for video */ id->i_channels = 0; - id->i_port = i_port; if( p_fmt != NULL ) { id->i_cat = p_fmt->i_cat; @@ -979,7 +944,7 @@ static sout_stream_id_t *Add( sout_stream_t *p_stream, es_format_t *p_fmt ) id->i_bitrate = 0; } - id->i_mtu = config_GetInt( p_stream, "mtu" ); + id->i_mtu = var_InheritInteger( p_stream, "mtu" ); if( id->i_mtu <= 12 + 16 ) id->i_mtu = 576 - 20 - 8; /* pessimistic */ msg_Dbg( p_stream, "maximum RTP packet size: %d bytes", id->i_mtu ); @@ -988,7 +953,18 @@ static sout_stream_id_t *Add( sout_stream_t *p_stream, es_format_t *p_fmt ) #ifdef HAVE_SRTP id->srtp = NULL; +#endif + vlc_mutex_init( &id->lock_sink ); + id->sinkc = 0; + id->sinkv = NULL; + id->rtsp_id = NULL; + id->p_fifo = NULL; + id->listen.fd = NULL; + + id->i_caching = + (int64_t)1000 * var_GetInteger( p_stream, SOUT_CFG_PREFIX "caching"); +#ifdef HAVE_SRTP char *key = var_CreateGetNonEmptyString (p_stream, SOUT_CFG_PREFIX"key"); if (key) { @@ -1013,19 +989,48 @@ static sout_stream_id_t *Add( sout_stream_t *p_stream, es_format_t *p_fmt ) } #endif - vlc_mutex_init( &id->lock_sink ); - id->sinkc = 0; - id->sinkv = NULL; - id->rtsp_id = NULL; - id->p_fifo = NULL; - id->listen.fd = NULL; - - id->i_caching = - (int64_t)1000 * var_GetInteger( p_stream, SOUT_CFG_PREFIX "caching"); + id->i_seq_sent_next = id->i_sequence; if( p_sys->psz_destination != NULL ) + { + /* Choose the port */ + uint16_t i_port = 0; + if( p_fmt == NULL ) + ; + else + if( p_fmt->i_cat == AUDIO_ES && p_sys->i_port_audio > 0 ) + i_port = p_sys->i_port_audio; + else + if( p_fmt->i_cat == VIDEO_ES && p_sys->i_port_video > 0 ) + i_port = p_sys->i_port_video; + + /* We do not need the ES lock (p_sys->lock_es) here, because + * this is the only one thread that can *modify* the ES table. + * The ES lock protects the other threads from our modifications + * (TAB_APPEND, TAB_REMOVE). */ + for (int i = 0; i_port && (i < p_sys->i_es); i++) + if (i_port == p_sys->es[i]->i_port) + i_port = 0; /* Port already in use! */ + for (uint16_t p = p_sys->i_port; i_port == 0; p += 2) + { + if (p == 0) + { + msg_Err (p_stream, "too many RTP elementary streams"); + goto error; + } + i_port = p; + for (int i = 0; i_port && (i < p_sys->i_es); i++) + if (p == p_sys->es[i]->i_port) + i_port = 0; + } + + id->i_port = i_port; + + int type = SOCK_STREAM; + switch( p_sys->proto ) { +#ifdef SOCK_DCCP case IPPROTO_DCCP: { const char *code; @@ -1037,11 +1042,13 @@ static sout_stream_id_t *Add( sout_stream_t *p_stream, es_format_t *p_fmt ) default: code = "RTPORTPV"; break; } var_SetString (p_stream, "dccp-service", code); + type = SOCK_DCCP; } /* fall through */ +#endif case IPPROTO_TCP: id->listen.fd = net_Listen( VLC_OBJECT(p_stream), p_sys->psz_destination, i_port, - p_sys->proto ); + type, p_sys->proto ); if( id->listen.fd == NULL ) { msg_Err( p_stream, "passive COMEDIA RTP socket failed" ); @@ -1073,6 +1080,7 @@ static sout_stream_id_t *Add( sout_stream_t *p_stream, es_format_t *p_fmt ) rtp_add_sink( id, fd, p_sys->rtcp_mux, NULL ); } } + } if( p_fmt == NULL ) { @@ -1189,34 +1197,59 @@ static sout_stream_id_t *Add( sout_stream_t *p_stream, es_format_t *p_fmt ) char *p_64_pps = NULL; char hexa[6+1]; - while( i_buffer > 4 && - p_buffer[0] == 0 && p_buffer[1] == 0 && - p_buffer[2] == 0 && p_buffer[3] == 1 ) + while( i_buffer > 4 ) { - const int i_nal_type = p_buffer[4]&0x1f; - int i_offset; + int i_offset = 0; int i_size = 0; + while( p_buffer[0] != 0 || p_buffer[1] != 0 || + p_buffer[2] != 1 ) + { + p_buffer++; + i_buffer--; + if( i_buffer == 0 ) break; + } + + if( i_buffer < 4 || memcmp(p_buffer, "\x00\x00\x01", 3 ) ) + { + msg_Dbg( p_stream, "No startcode found.."); + break; + } + p_buffer += 3; + i_buffer -= 3; + + const int i_nal_type = p_buffer[0]&0x1f; + msg_Dbg( p_stream, "we found a startcode for NAL with TYPE:%d", i_nal_type ); i_size = i_buffer; - for( i_offset = 4; i_offset+3 < i_buffer ; i_offset++) + for( i_offset = 0; i_offset+2 < i_buffer ; i_offset++) { - if( !memcmp (p_buffer + i_offset, "\x00\x00\x00\x01", 4 ) ) + if( !memcmp(p_buffer + i_offset, "\x00\x00\x01", 3 ) ) { /* we found another startcode */ + while( i_offset > 0 && 0 == p_buffer[ i_offset - 1 ] ) + i_offset--; i_size = i_offset; break; } } + + if( i_size == 0 ) + { + msg_Dbg( p_stream, "No-info found in nal "); + continue; + } + if( i_nal_type == 7 ) { - p_64_sps = vlc_b64_encode_binary( &p_buffer[4], i_size - 4 ); - sprintf_hexa( hexa, &p_buffer[5], 3 ); + p_64_sps = vlc_b64_encode_binary( p_buffer, i_size ); + /* XXX: nothing ensures that i_size >= 4 ?? */ + sprintf_hexa( hexa, &p_buffer[1], 3 ); } else if( i_nal_type == 8 ) { - p_64_pps = vlc_b64_encode_binary( &p_buffer[4], i_size - 4 ); + p_64_pps = vlc_b64_encode_binary( p_buffer, i_size ); } i_buffer -= i_size; p_buffer += i_size; @@ -1237,12 +1270,11 @@ static sout_stream_id_t *Add( sout_stream_t *p_stream, es_format_t *p_fmt ) case VLC_CODEC_MP4V: { - char hexa[2*p_fmt->i_extra +1]; - id->psz_enc = "MP4V-ES"; id->pf_packetize = rtp_packetize_split; if( p_fmt->i_extra > 0 ) { + char hexa[2*p_fmt->i_extra +1]; sprintf_hexa( hexa, p_fmt->p_extra, p_fmt->i_extra ); if( asprintf( &id->psz_fmtp, "profile-level-id=3; config=%s;", hexa ) == -1 ) @@ -1322,7 +1354,7 @@ static sout_stream_id_t *Add( sout_stream_t *p_stream, es_format_t *p_fmt ) } if (id->i_payload_type >= 96) /* Mark dynamic payload type in use */ - p_sys->payload_bitmap |= 1 << (id->i_payload_type - 96); + p_sys->payload_bitmap &= ~(1 << (127 - id->i_payload_type)); #if 0 /* No payload formats sets this at the moment */ int cscov = -1; @@ -1339,15 +1371,20 @@ static sout_stream_id_t *Add( sout_stream_t *p_stream, es_format_t *p_fmt ) id->i_ts_offset = rtp_compute_ts( id, p_sys->i_pts_offset ); if( p_sys->rtsp != NULL ) - id->rtsp_id = RtspAddId( p_sys->rtsp, id, p_sys->i_es, + id->rtsp_id = RtspAddId( p_sys->rtsp, id, GetDWBE( id->ssrc ), p_sys->psz_destination, p_sys->i_ttl, id->i_port, id->i_port + 1 ); id->p_fifo = block_FifoNew(); - if( vlc_thread_create( id, "RTP send thread", ThreadSend, - VLC_THREAD_PRIORITY_HIGHEST ) ) + if( unlikely(id->p_fifo == NULL) ) + goto error; + if( vlc_clone( &id->thread, ThreadSend, id, VLC_THREAD_PRIORITY_HIGHEST ) ) + { + block_FifoRelease( id->p_fifo ); + id->p_fifo = NULL; goto error; + } /* Update p_sys context */ vlc_mutex_lock( &p_sys->lock_es ); @@ -1378,33 +1415,35 @@ static int Del( sout_stream_t *p_stream, sout_stream_id_t *id ) { sout_stream_sys_t *p_sys = p_stream->p_sys; - if( id->p_fifo != NULL ) - { - vlc_object_kill( id ); - vlc_thread_join( id ); - block_FifoRelease( id->p_fifo ); - } - vlc_mutex_lock( &p_sys->lock_es ); TAB_REMOVE( p_sys->i_es, p_sys->es, id ); vlc_mutex_unlock( &p_sys->lock_es ); + if( likely(id->p_fifo != NULL) ) + { + vlc_cancel( id->thread ); + vlc_join( id->thread, NULL ); + block_FifoRelease( id->p_fifo ); + } + /* Release dynamic payload type */ if (id->i_payload_type >= 96) - p_sys->payload_bitmap &= ~(1 << (id->i_payload_type - 96)); + p_sys->payload_bitmap |= 1 << (127 - id->i_payload_type); free( id->psz_fmtp ); if( id->rtsp_id ) RtspDelId( p_sys->rtsp, id->rtsp_id ); - if( id->sinkc > 0 ) - rtp_del_sink( id, id->sinkv[0].rtp_fd ); /* sink for explicit dst= */ if( id->listen.fd != NULL ) { vlc_cancel( id->listen.thread ); vlc_join( id->listen.thread, NULL ); net_ListenClose( id->listen.fd ); } + /* Delete remaining sinks (incoming connections or explicit + * outgoing dst=) */ + while( id->sinkc > 0 ) + rtp_del_sink( id, id->sinkv[0].rtp_fd ); #ifdef HAVE_SRTP if( id->srtp != NULL ) srtp_destroy( id->srtp ); @@ -1416,8 +1455,7 @@ static int Del( sout_stream_t *p_stream, sout_stream_id_t *id ) if( p_sys->b_export_sap && !p_sys->p_mux ) SapSetup( p_stream ); if( p_sys->psz_sdp_file != NULL ) FileSetup( p_stream ); - vlc_object_detach( id ); - vlc_object_release( id ); + free( id ); return VLC_SUCCESS; } @@ -1480,7 +1518,7 @@ static int FileSetup( sout_stream_t *p_stream ) if( p_sys->psz_sdp == NULL ) return VLC_EGENERIC; /* too early */ - if( ( f = utf8_fopen( p_sys->psz_sdp_file, "wt" ) ) == NULL ) + if( ( f = vlc_fopen( p_sys->psz_sdp_file, "wt" ) ) == NULL ) { msg_Err( p_stream, "cannot open file '%s' (%m)", p_sys->psz_sdp_file ); @@ -1548,7 +1586,7 @@ static int HttpCallback( httpd_file_sys_t *p_args, /**************************************************************************** * RTP send ****************************************************************************/ -static void* ThreadSend( vlc_object_t *p_this ) +static void* ThreadSend( void *data ) { #ifdef WIN32 # define ECONNREFUSED WSAECONNREFUSED @@ -1560,7 +1598,7 @@ static void* ThreadSend( vlc_object_t *p_this ) # define EAGAIN WSAEWOULDBLOCK # define EWOULDBLOCK WSAEWOULDBLOCK #endif - sout_stream_id_t *id = (sout_stream_id_t *)p_this; + sout_stream_id_t *id = data; unsigned i_caching = id->i_caching; for (;;) @@ -1581,7 +1619,7 @@ static void* ThreadSend( vlc_object_t *p_this ) if( val ) { errno = val; - msg_Dbg( id, "SRTP sending error: %m" ); + msg_Dbg( id->p_stream, "SRTP sending error: %m" ); block_Release( out ); out = NULL; } @@ -1641,7 +1679,7 @@ static void* ThreadSend( vlc_object_t *p_this ) for( unsigned i = 0; i < deadc; i++ ) { - msg_Dbg( id, "removing socket %d", deadv[i] ); + msg_Dbg( id->p_stream, "removing socket %d", deadv[i] ); rtp_del_sink( id, deadv[i] ); } vlc_restorecancel (canc); @@ -1659,7 +1697,7 @@ static void *rtp_listen_thread( void *data ) for( ;; ) { - int fd = net_Accept( id, id->listen.fd ); + int fd = net_Accept( id->p_stream, id->listen.fd ); if( fd == -1 ) continue; int canc = vlc_savecancel( ); @@ -1677,7 +1715,7 @@ int rtp_add_sink( sout_stream_id_t *id, int fd, bool rtcp_mux, uint16_t *seq ) sink.rtcp = OpenRTCP( VLC_OBJECT( id->p_stream ), fd, IPPROTO_UDP, rtcp_mux ); if( sink.rtcp == NULL ) - msg_Err( id, "RTCP failed!" ); + msg_Err( id->p_stream, "RTCP failed!" ); vlc_mutex_lock( &id->lock_sink ); INSERT_ELEM( id->sinkv, id->sinkc, id->sinkc, sink ); @@ -1740,25 +1778,6 @@ int64_t rtp_get_ts( const sout_stream_t *p_stream ) return p_sys->i_pts_zero + (now - i_npt_zero); } -/* FIXME: this is pretty bad - if we remove and then insert an ES - * the number will get unsynched from inside RTSP */ -unsigned rtp_get_num( const sout_stream_id_t *id ) -{ - sout_stream_sys_t *p_sys = id->p_stream->p_sys; - int i; - - vlc_mutex_lock( &p_sys->lock_es ); - for( i = 0; i < p_sys->i_es; i++ ) - { - if( id == p_sys->es[i] ) - break; - } - vlc_mutex_unlock( &p_sys->lock_es ); - - return i; -} - - void rtp_packetize_common( sout_stream_id_t *id, block_t *out, int b_marker, int64_t i_pts ) {