[FFmpeg-devel] [PATCH] don't trust avi riff tag end

[Baptiste Coudurier]

Michael Niedermayer wrote:
> On Thu, Mar 19, 2009 at 11:32:32AM -0700, Baptiste Coudurier wrote:
>> Hi,
>>
>> On 3/18/2009 9:39 PM, Michael Niedermayer wrote:
>>> On Wed, Mar 18, 2009 at 06:35:21PM -0700, Baptiste Coudurier wrote:
>>>> Hi,
>>>>
>>>> $subject,
>>>>
>>>> This should fix demuxing avi generated through streaming and read
>>>> through streaming.
>>>>
>>>> In the later case, RIFF tag end is 0, and when reading url_fsize will
>>>> return 0.
>>>>
>>>> Like ffmpeg -i <file> -f avi - | ffmpeg -i -
>>>>
>>>> Any problem with this ?
>>> fsize is used to detect damaged chunks, with INT64_MAX damaged chunks
>>> are accepted and can lead to skiping of the whole file instead of searching
>>> for the next valid chunk
>> I see, would it be ok to only set fsize to INT64_MAX when riff end tag
>> is not set then ?
> 
> yes
> 

Patch attached.

-- 
Baptiste COUDURIER                              GnuPG Key Id: 0x5C1ABAAA
Key fingerprint                 8D77134D20CC9220201FC5DB0AC9325C5C1ABAAA
checking for life_signs in -lkenny... no
FFmpeg maintainer                                  http://www.ffmpeg.org
-------------- next part --------------
A non-text attachment was scrubbed...
Name: fix_pipe_avi.patch
Type: text/x-diff
Size: 413 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20090319/43174886/attachment.patch>