[FFmpeg-devel] Fwd: [PATCH] Psygnosis YOP demuxer

[Michael Niedermayer]

On Tue, Jan 05, 2010 at 11:03:29PM +0530, Mohamed Naufal wrote:
> 2010/1/5 Michael Niedermayer <michaelni at gmx.at>
> 
> 
> [...]
> 
> 
> > > +static int yop_read_packet(AVFormatContext *s, AVPacket *pkt)
> > > +{
> > > +    YopDecContext *yop = s->priv_data;
> > > +    ByteIOContext *pb  = s->pb;
> > > +
> > > +    int ret;
> > > +    int palette_size           = 4 + yop->num_pal_colors * 3;
> >
> > > +    int actual_video_data_size = yop->frame_size -
> > > +                                 yop->audio_block_length - palette_size;
> >
> > this still can end up negative, there still are missing checks in the
> > header parsing
> >
> >
> 
> Fixed.
> 
> [...]
> 
> Naufal

>  Changelog                |    1 
>  doc/general.texi         |    1 
>  libavformat/Makefile     |    1 
>  libavformat/allformats.c |    1 
>  libavformat/avformat.h   |    2 
>  libavformat/yop.c        |  186 +++++++++++++++++++++++++++++++++++++++++++++++
>  6 files changed, 191 insertions(+), 1 deletion(-)
> 05b7713388599f9ce9d51388deb0b792ae89051f  yop_demux.patch

patch ok if it works (also seeking and no crashes with damaged files)

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

When you are offended at any man's fault, turn to yourself and study your
own failings. Then you will forget your anger. -- Epictetus
-------------- 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/20100106/647ecf50/attachment.pgp>