Hi,

On Thu, Aug 4, 2011 at 1:50 PM, Reinhard Tartler <[email protected]> wrote:
> On Thu, Aug 04, 2011 at 22:33:54 (CEST), Ronald S. Bultje wrote:
>> On Thu, Aug 4, 2011 at 1:28 PM, Reinhard Tartler <[email protected]> wrote:
>>> From: Chris Evans <[email protected]>
>>>
>>> Prevents a crash with certain mp4 files.
>>>
>>> References:
>>> http://code.google.com/p/chromium/issues/detail?id=7302
>>> http://codereview.chromium.org/6597005
>>> http://cve.mitre.org/cgi-bin/cvename.cgi?name=CVE-2011-1198
>>> ---
>>>  libavcodec/pthread.c |   14 ++++++++------
>>>  1 files changed, 8 insertions(+), 6 deletions(-)
>>
>> I've seen this patch. I don't think it is appliccable to current
>> git/master, because I couldn't reproduce the crash.
>>
>> I think previous versions had additional return paths that were since
>> then removed.
>
> Does it apply to 0.7? Or do you believe that it is chrome/ffmpeg-mt
> specific?

Libav (or FFmpeg) never had that return path, so I think we can safely
ignore the patch, unless someone can reproduce a crash (or valgrind
error, or anything unusual) with the file from that bug report. I
couldn't.

> I *guess* it doesn't apply to 0.6 and earlier, because we didn't have mt
> back then.

Correct.

Ronald
_______________________________________________
libav-devel mailing list
[email protected]
https://lists.libav.org/mailman/listinfo/libav-devel

Reply via email to