[FFmpeg-devel] [PATCH] Fix VC-1 width/height handling.

Reimar Döffinger Reimar.Doeffinger at gmx.de
Wed Aug 17 01:01:56 CEST 2011


On Tue, Aug 16, 2011 at 04:13:31AM +0200, Michael Niedermayer wrote:
> On Sat, Aug 13, 2011 at 11:56:20AM +0200, Reimar Döffinger wrote:
> > avcodec_set_dimensions should be used for size changes to ensure
> > compatibility with future changes.
> > avctx->width/avctx->height may not be set to display-only dimensions.
> > Even more so since vc1dec.c would later set coded_width/height based
> > on this.
> > coded_width/coded_height should be used instead of width/height for
> > decoder setup.
> > This fixes playback of e.g. zz-mcr-nsqr.vc1 sample (containing
> > display width/height settings) in MPlayer and should fix a crash
> > with MPC: http://forum.doom9.org/showthread.php?t=162221.
> > 
> > Signed-off-by: Reimar Döffinger <Reimar.Doeffinger at gmx.de>
> > ---
> >  libavcodec/vc1.c    |   31 +++++++++++++++++--------------
> >  libavcodec/vc1dec.c |   10 ++++------
> >  2 files changed, 21 insertions(+), 20 deletions(-)
> 
> should be ok

Pushed.


More information about the ffmpeg-devel mailing list