[FFmpeg-devel] Realmedia patch

[Ronald S. Bultje]

Hi Luca,

On Fri, Aug 22, 2008 at 2:08 PM, Luca Abeni <lucabe72 at email.it> wrote:
> Ronald S. Bultje wrote:
>> Index: ffmpeg/libavformat/rtsp.c
>> ===================================================================
>> --- ffmpeg.orig/libavformat/rtsp.c    2008-07-03 08:46:52.000000000 -0400
>> +++ ffmpeg/libavformat/rtsp.c 2008-07-03 08:48:34.000000000 -0400
[..]
>> +    /* request options supported by the server */
>> +    for (is_real_datatype = 0;; is_real_datatype++) {
>> +        snprintf(cmd, sizeof(cmd),
>> +                 "OPTIONS %s RTSP/1.0\r\n", s->filename);
>> +        if (is_real_datatype)
>> +            av_strlcat(cmd,
>> +                       "User-Agent: RealMedia Player Version 6.0.9.1235 (linux-2.0-libc6-i386-gcc2.95\r\n"
>> +                       "ClientChallenge: 9e26d33f2984236010ef6253fb1887f7\r\n"
>> +                       "PlayerStarttime: [28/03/2003:22:50:23 00:00]\r\n"
>> +                       "CompanyID: KnKV4M4I/B2FjJ1TToLycw==\r\n"
>> +                       "GUID: 00000000-0000-0000-0000-000000000000\r\n"
>> +                       "RegionData: 0\r\n"
>> +                       "ClientID: Linux_2.4_6.0.9.1235_play32_RN01_EN_586\r\n",
>> +                       sizeof(cmd));
>
> Is this really needed? I mean: claiming to be RealMedia Player, sending
> a random Starttime, etc...
> Are all these tags needed? (what happens if you remove ClientID,
> User-Agent, etc...?)
>
> (anyway, I think that the usage of the "OPTIONS" command to identify the
> server can be committed independently from the rest of the patch)

Attached is a patch that implements just OPTIONS. I'll send a few more
that implement RDT-compatible DESCRIBE and SETUP.

Ronald
-------------- next part --------------
An embedded and charset-unspecified text was scrubbed...
Name: rtsp-options.patch
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20080824/cd2290cd/attachment.txt>