[FFmpeg-devel] [PATCH] vp9: reinit internal buffers on bpp change.

Ronald S. Bultje rsbultje at gmail.com
Tue Dec 1 16:06:22 CET 2015


Hi,

On Tue, Dec 1, 2015 at 9:21 AM, Hendrik Leppkes <h.leppkes at gmail.com> wrote:

> On Tue, Dec 1, 2015 at 3:13 PM, Ronald S. Bultje <rsbultje at gmail.com>
> wrote:
> > Fixes mozilla bug 1229128.
> > ---
> >  libavcodec/vp9.c | 3 ++-
> >  1 file changed, 2 insertions(+), 1 deletion(-)
> >
> > diff --git a/libavcodec/vp9.c b/libavcodec/vp9.c
> > index d4061e2..858d561 100644
> > --- a/libavcodec/vp9.c
> > +++ b/libavcodec/vp9.c
> > @@ -219,7 +219,8 @@ static int update_size(AVCodecContext *ctx, int w,
> int h, enum AVPixelFormat fmt
> >
> >      av_assert0(w > 0 && h > 0);
> >
> > -    if (s->intra_pred_data[0] && w == ctx->width && h == ctx->height &&
> ctx->pix_fmt == fmt)
> > +    if (s->intra_pred_data[0] && w == ctx->width && h == ctx->height &&
> > +        ctx->pix_fmt == fmt && s->bpp == s->last_bpp)
> >          return 0;
>
> Doesn't the pixfmt change when bitdepth changes?


Hm, yes, let me dig some deeper, I think there's something more shitty
hiding in there...

Patch withdrawn, for now.

Ronald


More information about the ffmpeg-devel mailing list