[FFmpeg-devel] [PATCH] MLP/TrueHD decoder

Michael Niedermayer michaelni
Wed Nov 14 12:10:19 CET 2007


On Wed, Nov 14, 2007 at 10:49:16AM +0000, Ian Caulfield wrote:
> Hi,
> 
> On Nov 14, 2007 10:28 AM, Michael Niedermayer <michaelni at gmx.at> wrote:
> 
> >
> > it can be split over many more than 2
> > and ff_combine_frame() should deal with that or did i misunderstand you
> > and theres another problem with that?
> 
> 
> The main problem I could see was that if (for whatever reason) the parser
> gets fed with garbage and no sync word is found, then it'll eat up memory as
> it stashes data that won't be used anyway... or am I being dense?

well add a
if(size to big) return packet

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

I do not agree with what you have to say, but I'll defend to the death your
right to say it. -- Voltaire
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: Digital signature
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20071114/25c3c816/attachment.pgp>



More information about the ffmpeg-devel mailing list