[FFmpeg-devel] [PATCH] Fix crash in libx264 when rc_eq is not set

Stefano Sabatini stefano.sabatini-lala
Mon Aug 25 18:03:57 CEST 2008


On date Monday 2008-08-25 17:41:05 +0200, Guillaume POIRIER encoded:
> Hello,
> 
> On Mon, Aug 25, 2008 at 5:19 PM, Stefano Sabatini
> <stefano.sabatini-lala at poste.it> wrote:
> 
> [..]
> >> Index: libavcodec/libx264.c
> >> ===================================================================
> >> --- libavcodec/libx264.c      (revision 14863)
> >> +++ libavcodec/libx264.c      (working copy)
> >> @@ -253,7 +253,7 @@
> >>      x4->params.rc.f_ip_factor = 1/fabs(avctx->i_quant_factor);
> >>      x4->params.rc.f_pb_factor = avctx->b_quant_factor;
> >>      x4->params.analyse.i_chroma_qp_offset = avctx->chromaoffset;
> >> -    x4->params.rc.psz_rc_eq = avctx->rc_eq;
> >> +    x4->params.rc.psz_rc_eq = avctx->rc_eq ? avctx->rc_eq : "tex^qComp";
> >>
> >>      x4->params.analyse.b_psnr = avctx->flags & CODEC_FLAG_PSNR;
> >>      x4->params.i_log_level = X264_LOG_DEBUG;
> >
> > Ping?
> >
> > A lot of people are complainig about this on ffmpeg-user, could
> > someone review it?
> 
> no need to. x264 won't need rc_eq to be set in a few days...

But older libx264 versions will lye around for a long time, so IMO is
still a good idea to fix the problem.

Regards.
-- 
FFmpeg = Fast and Faboulous Multimedia Pitiless EnGine




More information about the ffmpeg-devel mailing list