(Another bug fixed due to the great new apple-bugreport mailinglist)

It seems I had never committed this and then checked out a master version of it or something when working on libass tree.

* Check asprintf return value
* Use asprintf instead of malloc/sprintf (less dangerous)

This fixes issues with files which had their Cues referenced from this 2nd Seekhead at the end of the file. These files were previously treated as "broken".
A maximum of 10 seek heads is allowed in our module, to prevent recursion. The module is much more flexible now, but not that some Master elements that are allowed to be present multiple times are still handled only once (KaxInfo, KaxTracks, KaxTags).

With these change, most mkv files produced with mkvmerge will produce a significant amount of debug. This is because these files have a load of Junk tags in their eof Seekhead that our parser is warning about. After consulting with robux, it was determined that this was a bug in mkvmerge and a ticket has been filed ( https://www.bunkus.org/bugzilla/show_bug.cgi?id=290 ).

This fixes #1236

(I wonder how we can have so stupid code)

(I don't know if this is exploitable - depends what libpng does)

This is most likely exploitable with malicious sub files.

Do not do this with portable code. Ever.
This reverts commit b204087e.

Signed-off-by: Jean-Baptiste Kempf <jb@videolan.org>