[FFmpeg-devel] [PATCH]: fix ES name in the UMF media description

[Thierry Foucu]

On Wed, Jul 14, 2010 at 11:18 AM, Thierry Foucu <tfoucu at gmail.com> wrote:

>
>
> On Wed, Jul 14, 2010 at 11:01 AM, Baptiste Coudurier <
> baptiste.coudurier at gmail.com> wrote:
>
>> On 07/14/2010 10:48 AM, Thierry Foucu wrote:
>>
>>> On Wed, Jul 14, 2010 at 10:47 AM, Thierry Foucu<tfoucu at gmail.com>
>>>  wrote:
>>> [...]
>>>
>>>
>>> Index: libavformat/gxfenc.c
>>> ===================================================================
>>> --- libavformat/gxfenc.c        (revision 24229)
>>> +++ libavformat/gxfenc.c        (working copy)
>>> @@ -550,9 +550,9 @@
>>>          put_le32(pb, 0); /* attributes rw, ro */
>>>          put_le32(pb, 0); /* mark in */
>>>          put_le32(pb, gxf->nb_fields); /* mark out */
>>> -        put_buffer(pb, ES_NAME_PATTERN, sizeof(ES_NAME_PATTERN));
>>> +        put_buffer(pb, ES_NAME_PATTERN, strlen(ES_NAME_PATTERN));
>>>          put_be16(pb, sc->media_info);
>>> -        for (j = sizeof(ES_NAME_PATTERN)+2; j<  88; j++)
>>> +        for (j = strlen(ES_NAME_PATTERN)+2; j<  88; j++)
>>>              put_byte(pb, 0);
>>>          put_le32(pb, sc->track_type);
>>>          put_le32(pb, sc->sample_rate);
>>>
>>
>> Patch ok.
>>
>>
> Thanks. When you have time, could you apply it as I do not have write
> access.
>

not applied?
Just FYI, without this change, the gxf file is not compatible with any Grass
Valley decoder. They cannot find the ES in it.


>
>
>>  --
>> Baptiste COUDURIER
>> Key fingerprint                 8D77134D20CC9220201FC5DB0AC9325C5C1ABAAA
>> FFmpeg maintainer                                  http://www.ffmpeg.org
>>
>> _______________________________________________
>> ffmpeg-devel mailing list
>> ffmpeg-devel at mplayerhq.hu
>> https://lists.mplayerhq.hu/mailman/listinfo/ffmpeg-devel
>>
>
>