[FFmpeg-devel] [PATCH] robustify parsing of flv metadata

Pascal Massimino pascal.massimino
Tue Mar 18 23:02:04 CET 2008


 Hi,

On Tue, Mar 18, 2008 at 2:53 PM, Pascal Massimino <
pascal.massimino at gmail.com> wrote:

>   Michael,
>
> On Tue, Mar 18, 2008 at 2:44 PM, Michael Niedermayer <michaelni at gmx.at>
> wrote:
>
> > On Tue, Mar 18, 2008 at 02:33:24PM -0700, Pascal Massimino wrote:
> > >   Gents,
> > >
> > >  a little patch to prevent non-sense num_val in broken
> > >  metadata packets. Maybe it's better to bail out in such
> > >  cases, dunno for sure.
> >
> > Doesnt work, num_val is double and can after casting to int still be
> > nonsense.
>
>  oh indeed! we should be testing ((int)num_val >= 0) instead...
>
> >
> > Also thanks for the sample to test unwrap-diff ;)
>
>
> ok ok, will prepare an updated patch and be a good attaching citizen :)
>

here, *attached*

skal
-------------- next part --------------
A non-text attachment was scrubbed...
Name: flvdec.patch
Type: text/x-diff
Size: 1443 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20080318/291ff684/attachment.patch>



More information about the ffmpeg-devel mailing list