[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