[FFmpeg-devel] [PATCH] avfilter/vf_transpose: Fix regression with packed pixel formats

[Michael Niedermayer]

On Sun, Jan 28, 2018 at 10:12:19AM +0100, Paul B Mahol wrote:
> On 1/28/18, Michael Niedermayer <michael at niedermayer.cc> wrote:
> > Regression since: c6939f65a116b1ffed345d29d8621ee4ffb32235
> > Found-by: Paul B Mahol <onemda at gmail.com>
> > Signed-off-by: Michael Niedermayer <michael at niedermayer.cc>
> > ---
> >  libavfilter/vf_transpose.c | 2 +-
> >  1 file changed, 1 insertion(+), 1 deletion(-)
> >
> > diff --git a/libavfilter/vf_transpose.c b/libavfilter/vf_transpose.c
> > index 1e1a5c4b89..bf2ab7eb18 100644
> > --- a/libavfilter/vf_transpose.c
> > +++ b/libavfilter/vf_transpose.c
> > @@ -217,7 +217,7 @@ static int config_props_output(AVFilterLink *outlink)
> >
> >      s->hsub = desc_in->log2_chroma_w;
> >      s->vsub = desc_in->log2_chroma_h;
> > -    s->planes = desc_in->nb_components;
> > +    s->planes = (desc_in->flags & AV_PIX_FMT_FLAG_PLANAR) ?
> > desc_in->nb_components : 1;
> >
> >      av_assert0(desc_in->nb_components == desc_out->nb_components);
> >
> > --
> > 2.16.1
> >
> > _______________________________________________
> > ffmpeg-devel mailing list
> > ffmpeg-devel at ffmpeg.org
> > http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
> >
> 
> There is function that count number of planes.

yes, just realized that too, i will change the code to use that

thx

[...]
-- 
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: signature.asc
Type: application/pgp-signature
Size: 181 bytes
Desc: not available
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20180128/fc21c1c8/attachment.sig>