[FFmpeg-devel] [PATCH]Fix invalid memory accesses using the fade filter

[Michael Niedermayer]

On Tue, Feb 24, 2015 at 11:41:02AM +0100, Carl Eugen Hoyos wrote:
> On Monday 23 February 2015 02:02:42 pm Clément Bœsch wrote:
> > On Mon, Feb 23, 2015 at 01:59:45PM +0100, Michael Niedermayer wrote:
> > > On Mon, Feb 23, 2015 at 03:27:54AM +0100, Carl Eugen Hoyos wrote:
> 
> > > > +        int width = av_pix_fmt_desc_get(frame->format)->flags &
> > > > AV_PIX_FMT_FLAG_PLANAR ? +                    frame->width :
> > > > +                    frame->width * s->bpp;
> > > > -        for (j = 0; j < frame->width * s->bpp; j++) {
> > > > +        for (j = 0; j < width; j++) {
> > >
> > > should be ok, alternatively bpp could be renamed and set to 1 for
> > > planar
> >
> > And moved out of the loop
> 
> New patch attached.

ok if tested

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

He who knows, does not speak. He who speaks, does not know. -- Lao Tsu
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 181 bytes
Desc: Digital signature
URL: <https://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20150224/a936d6ed/attachment.asc>