[FFmpeg-devel] fix for >2GB files in flvdec.c

Pascal Massimino pascal.massimino
Mon Oct 20 19:30:14 CEST 2008


  Hey Baptiste,

On Mon, Oct 20, 2008 at 10:08 AM, Baptiste Coudurier
<baptiste.coudurier at smartjog.com> wrote:
> Hi Pascal,
>
> Pascal Massimino wrote:
>> Hi everybody,
>>
>> patch against r15655 attached...
>>
>  > [...]
>>
>  > @@ -317,7 +318,7 @@
>>      if(size == 0)
>>          continue;
>>
>> -    next= size + url_ftell(s->pb);
>> +    next= url_ftell(s->pb) + size;
>
> Cosmetics ?

well, actually i'm not sure: will 'size' be promoted int64 if it's
first? or will the int64 in url_ftell() be demoted 'int' because of
the 'size' in front (yikes!)?
In doubt, i rely on left-to-right prevalence rule, but that's maybe just me...

>
>>      if (type == FLV_TAG_TYPE_AUDIO) {
>>          is_audio=1;
>> @@ -370,8 +371,8 @@
>>      // if not streamed and no duration from metadata then seek to end to find the duration from the timestamps
>>      if(!url_is_streamed(s->pb) && s->duration==AV_NOPTS_VALUE){
>>          int size;
>> -        const int pos= url_ftell(s->pb);
>> -        const int fsize= url_fsize(s->pb);
>> +        const int64_t pos= url_ftell(s->pb);
>> +        const int64_t fsize= url_fsize(s->pb);
>
> Pos seems to be uselessly redeclared here, this can be commited
> separately I guess.
>

indeed, the top 'pos' variable could be localized to block 306-369.

skal


> Im not maintainer, Michael will confirm.
>
> --
> Baptiste COUDURIER                              GnuPG Key Id: 0x5C1ABAAA
> Smartjog USA Inc.                                http://www.smartjog.com
> Key fingerprint                 8D77134D20CC9220201FC5DB0AC9325C5C1ABAAA
> _______________________________________________
> ffmpeg-devel mailing list
> ffmpeg-devel at mplayerhq.hu
> https://lists.mplayerhq.hu/mailman/listinfo/ffmpeg-devel
>




More information about the ffmpeg-devel mailing list