[FFmpeg-devel] Realmedia patch

[Ronald S. Bultje]

Hi Luca,

On Wed, Sep 17, 2008 at 5:14 PM, Luca Abeni <lucabe72 at email.it> wrote:
> Ronald S. Bultje wrote:
> [...]
>> Index: ffmpeg-svn/libavformat/rtsp.c
>> ===================================================================
>> --- ffmpeg-svn.orig/libavformat/rtsp.c        2008-09-14 10:55:44.000000000 -0400
>> +++ ffmpeg-svn/libavformat/rtsp.c     2008-09-14 10:57:10.000000000 -0400
>> @@ -945,8 +945,11 @@
>>          rtp_opened:
>>              port = rtp_get_local_port(rtsp_st->rtp_handle);
>>              snprintf(transport, sizeof(transport) - 1,
>> -                     "%s/UDP;unicast;client_port=%d",
>> -                     trans_pref, port);
>> +                     "%s/UDP;", trans_pref);
>> +            if (rt->server_type != RTSP_SERVER_REAL)
>> +                av_strlcat(transport, "unicast;", sizeof(transport));
>> +            av_strlcatf(transport, sizeof(transport),
>> +                     "client_port=%d", port);
>>              if (rt->server_type == RTSP_SERVER_RTP)
>>                  av_strlcatf(transport, sizeof(transport), "-%d", port + 1);
>>          }
>
> The path itself is ok.
> But it enables RTD over UDP, and I am not sure if this combination
> is currently working. So, I'd suggest to apply the patch only after
> you tested that RDT over UDP works well.

I tested it, and it works just fine, not a single modification needed
anywhere. :-).

Ronald