[FFmpeg-devel] [PATCH] Workaround for MPEG-TS crashes

[Baptiste Coudurier]

On 9/15/09 1:03 PM, Baptiste Coudurier wrote:
> On 09/15/2009 12:17 PM, Joakim Plate wrote:
>> Baptiste Coudurier<baptiste.coudurier<at> gmail.com> writes:
>>
>>
>>>
>>> It has some overhead, however it is pretty robust. All IPTV feeds are
>>> basically mpeg-ts over multicast udp. 65k is reasonable indeed and
>>> should not cause problems.
>>>
>>
>> I must say i agree with micheal here. Why limit the amount if data
>> read to
>> resync. You could have corrupt data delivered by a DVB reciever for a
>> long
>> time. That doesn't mean it won't recover after a while.
>
> In any case the amount must be limited, demuxer must give up at a
> certain point to permit "reponsiveness".
>
>> av_log that you exceeded search amount. Return EAGAIN to libav user so
>> his
>> processing loop remains "responsive"
>
> Yes, it seems EAGAIN might be a good idea, if people are ok with not
> being able to distinguish "resync error" from "read again", for now.
>

Applied modifications.

-- 
Baptiste COUDURIER
Key fingerprint                 8D77134D20CC9220201FC5DB0AC9325C5C1ABAAA
FFmpeg maintainer                                  http://www.ffmpeg.org