[FFmpeg-devel] [PATCH] RTSP udp multicast support

[Michael Niedermayer]

Hi

On Mon, May 14, 2007 at 02:16:40PM +0200, Thijs Vermeir wrote:
> Hello,
> 
> It looks like the ffmpeg libs are not supporting different transport
> types for rtsp. (They are implemented but does not work with different
> encoders). With this patch you can subscribe to the rtsp multicast
> with the information from the sdp message.

[...]
> @@ -897,6 +899,9 @@
>  
>          rtsp_st = rt->rtsp_streams[i];
>  
> +        if ( strcmp(rtsp_st->sdp_transport, "RTP/AVP") == 0)
> +            protocol_mask = (1 << RTSP_PROTOCOL_RTP_UDP_MULTICAST);

i dont think parsing sdp strings belongs into rtsp_read_header()


> +
>          /* compute available transports */
>          transport[0] = '\0';
>  
> @@ -944,7 +949,7 @@
>                  pstrcat(transport, sizeof(transport), ",");
>              snprintf(transport + strlen(transport),
>                       sizeof(transport) - strlen(transport) - 1,
> -                     "RTP/AVP/UDP;multicast");
> +                     "RTP/AVP;multicast;client_port=%d-%d", rtsp_st->sdp_port, rtsp_st->sdp_port + 1);
>          }

why is this needed?

[...]
-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

No human being will ever know the Truth, for even if they happen to say it
by chance, they would not even known they had done so. -- Xenophanes
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20070704/54f263b1/attachment.pgp>