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

[Stefano Sabatini]

On date Monday 2008-08-25 12:21:25 -0600, Loren Merritt encoded:
> On Mon, 25 Aug 2008, M?ns Rullg?rd wrote:
>> Stefano Sabatini <stefano.sabatini-lala at poste.it> writes:
>>> On date Monday 2008-08-25 17:41:05 +0200, Guillaume POIRIER encoded:
>>>>
>>>> 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.
>>
>> But we need to support the latest x264, which won't even have that
>> field in the struct.  What does old/current x264 do if it isn't
>> touched at all?
>
> x264_param_default sets rc_eq to a sane value. The problem only occurs  
> when ffmpeg overrides that and sets it to null. So removing the line from 
> ffmpeg will work with all x264 versions.

Patch attached, OK to apply?

Regards.
-- 
FFmpeg = Foolish & Freak Multipurpose Portable Entertaining God
-------------- next part --------------
A non-text attachment was scrubbed...
Name: fix-libx64-rc-eq-crash-01.patch
Type: text/x-diff
Size: 542 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20080825/f433f7b9/attachment.patch>