[FFmpeg-cvslog] ffv1enc: Check the return value of ffv1_common_init()

[Michael Niedermayer]

On Sat, May 11, 2013 at 08:56:44PM +0200, Clément Bœsch wrote:
> On Sat, May 11, 2013 at 08:51:38PM +0200, Michael Niedermayer wrote:
> > ffmpeg | branch: master | Michael Niedermayer <michaelni at gmx.at> | Sat May 11 20:17:09 2013 +0200| [54602590d9e479e474db75cafa2e4b9f2799eafe] | committer: Michael Niedermayer
> > 
> > ffv1enc: Check the return value of ffv1_common_init()
> > 
> > Signed-off-by: Michael Niedermayer <michaelni at gmx.at>
> > 
> > > http://git.videolan.org/gitweb.cgi/ffmpeg.git/?a=commit;h=54602590d9e479e474db75cafa2e4b9f2799eafe
> > ---
> > 
> >  libavcodec/ffv1enc.c |    3 ++-
> >  1 file changed, 2 insertions(+), 1 deletion(-)
> > 
> > diff --git a/libavcodec/ffv1enc.c b/libavcodec/ffv1enc.c
> > index 79e483d..4535464 100644
> > --- a/libavcodec/ffv1enc.c
> > +++ b/libavcodec/ffv1enc.c
> > @@ -641,7 +641,8 @@ static av_cold int encode_init(AVCodecContext *avctx)
> >      const AVPixFmtDescriptor *desc = av_pix_fmt_desc_get(avctx->pix_fmt);
> >      int i, j, k, m, ret;
> >  
> > -    ffv1_common_init(avctx);
> > +    if ((ret = ffv1_common_init(avctx)) < ret)
> 
> Wut?
> 
> Don't you mean < 0?

yes, fixed

[...]
-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

Freedom in capitalist society always remains about the same as it was in
ancient Greek republics: Freedom for slave owners. -- Vladimir Lenin
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 198 bytes
Desc: Digital signature
URL: <http://ffmpeg.org/pipermail/ffmpeg-cvslog/attachments/20130511/38ee21ca/attachment.asc>