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

Guillaume POIRIER poirierg
Mon Aug 25 17:41:05 CEST 2008


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...

Guillaume
-- 
One should not give up hope on imbeciles. With a little training, you
can make them into soldiers.
 -- Pierre Desproges




More information about the ffmpeg-devel mailing list