On Sat, Dec 06, 2014 at 04:53:30PM +0100, wm4 wrote:
> When matroska_execute_seekhead() is called, it goes through the list of
> seekhead entries and attempts to read elements not read yet. When doing
> this, the parser can find further SeekHead elements, and will extend the
> matroska->seekhead list. This can lead to a (practically) infinite loop
> with certain broken files. (Maybe it can happen even with valid files.
> The demuxer doesn't seem to check correctly whether an element has
> already been read.)
> 
> Fix this by ignoring elements that were added to the seekhead field
> during executing seekhead entries.
> 
> This does not fix the possible situation when multiple SeekHead elements
> after the file header (i.e. occur after the "before_pos" file position)
> point to the same elements. These elements will probably be parsed
> multiple times, likely leading to bugs.
> 
> Fixes ticket #4162.
> ---
> I hope my analysis of the problem is correct.

applied

i also added a request for samples for this case (couldnt find any
except that fuzzed file)

thanks

[...]
-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

The educated differ from the uneducated as much as the living from the
dead. -- Aristotle 

Attachment: signature.asc
Description: Digital signature

_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

Reply via email to