[FFmpeg-devel] [PATCH] don't attempt to resync avi without actually having the 8 bytes needed

elupus elupus
Sat Oct 4 04:03:32 CEST 2008


On Sat, 4 Oct 2008 01:05:17 +0200, Michael Niedermayer wrote:

> On Fri, Oct 03, 2008 at 10:50:02PM +0200, elupus wrote:
>> [5 quoted lines suppressed]
> 
> your patch does not seem to change the behavior of the
> demuxer, it just makes it more complex
> 
> [...]

I've not seen any sample that requires this change. It was just a helper
when i was debugging the other issue, wasn't able to quickly decide if the
validation checks where guaranteed to fail on the 7 first read bytes. 

Now thinking about it, since it's a int and not a char, the values
generated by the memset is probably so way way off, that it will always
catch it. Still made it easier to debug since the validation checks would
actually log something usefull (if enabled).

Anyway i won't press it.

Joakim





More information about the ffmpeg-devel mailing list