[FFmpeg-devel] RTMP: trailing slash in content-base

[Ronald S. Bultje]

Hi,

On Wed, Feb 16, 2011 at 12:03 PM, Martin Storsj? <martin at martin.st> wrote:
> On Wed, 16 Feb 2011, Ronald S. Bultje wrote:
>
>> On Wed, Feb 16, 2011 at 9:39 AM, Martin Storsj? <martin at martin.st> wrote:
>> > But ffserver sure should be fixed to handle PLAY requests with a trailing
>> > slash I think, at least for compatibility with VLC/Live555. Does the
>> > attached patch work for you?
>> [..]
>> > @@ -3221,6 +3221,12 @@ static HTTPContext *find_rtp_session_with_url(const char *url,
>> > ? ? ? ? ?return rtp_c;
>> > ? ? ? ?}
>> > ? ? ?}
>> > + ? ?len = strlen(path);
>> > + ? ?if (path[len - 1] == '/') {
>> > + ? ? ? ?path[len - 1] = '\0';
>> > + ? ? ? ?if (!strcmp(path, rtp_c->stream->filename))
>> > + ? ? ? ? ? ?return rtp_c;
>> > + ? ?}
>> > ? ? ?return NULL;
>> > ?}
>>
>> len = strlen(path);
>> if (path[len-1]== '/' && !strncmp(path, rtp_c->stream->filename, len-1))
>> ? ? return rtp_c;
>>
>> Then you don't need to make path non-const, and then patch OK with me.
>
> Ah, true. New version attached. Nicolas, care to retest this version?

Patch OK with me if it works.

Ronald