[FFmpeg-devel] avcodec/proresdec : add 12b decoding support

Martin Vignali martin.vignali at gmail.com
Mon Nov 26 17:08:54 EET 2018


> > +    /* align height to 16, to avoid segfault */
> > +    tframe.f->height = FFALIGN(avctx->height, 16);
> > +    tframe.f->width = FFALIGN(avctx->width, 16);
> > +    tframe.f->crop_bottom = tframe.f->height - avctx->height;
> > +
> >      if ((ret = ff_thread_get_buffer(avctx, &tframe, 0)) < 0)
> >          return ret;
>
> Why is this now needed but was not before ?
> Is avcodec_align_dimensions2() correct ?
>
> Hello,

This is the part, where i'm not really sure about.
Like it's only segfault when there is alpha, but segfault on luma decoding
part and alpha decoding part.
Seems like, i need to add YUVA4xxP12 in avcodec_align_dimensions2, to have
the correct "height padding".

Will test this, and send new patch.

Thanks

Martin


More information about the ffmpeg-devel mailing list