On Mon, Feb 23, 2015 at 03:27:54AM +0100, Carl Eugen Hoyos wrote: > Hi! > > Attached patch fixes a crash with the following command line: > $ ffmpeg -loop 1 -i fate-suite/lena.pnm -vf format=yuva420p,fade -f null - > > Please comment, Carl Eugen
> vf_fade.c | 5 ++++-
> 1 file changed, 4 insertions(+), 1 deletion(-)
> 31297e0aecbb7ec4637c180ae16254499c628334 patchfade.diff
> diff --git a/libavfilter/vf_fade.c b/libavfilter/vf_fade.c
> index 80ce75d..5d012af 100644
> --- a/libavfilter/vf_fade.c
> +++ b/libavfilter/vf_fade.c
> @@ -203,7 +203,10 @@ static int filter_slice_luma(AVFilterContext *ctx, void
> *arg, int jobnr,
>
> for (i = slice_start; i < slice_end; i++) {
> uint8_t *p = frame->data[0] + i * frame->linesize[0];
> + 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
[...]
--
Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB
Why not whip the teacher when the pupil misbehaves? -- Diogenes of Sinope
signature.asc
Description: Digital signature
_______________________________________________ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
