[FFmpeg-devel] [PATCHv+ BUG REPORT] Various memory leaks

Vitor Sessak vitor1001
Tue Mar 2 22:33:51 CET 2010


Jai Menon wrote:
> On Sat, Feb 27, 2010 at 9:10 PM, Vitor Sessak <vitor1001 at gmail.com> wrote:
>> Vitor Sessak wrote:
>>> Hi,
>>>
>>> I was testing again FATE with valgrind. Attached is a fix for memory leaks
>>> in vp6 decoder and xa and dsicin demuxers. Also fails:
>>>
>>> Test 145 - Nullsoft Video (NSV) demuxer
>> Patch by Jai waiting review
> 
> I'm pretty sure its correct :) so maybe just commit and francois could
> flame on cvslog?

Looks good to me too. I think two sets of eyeballs for such a simple 
patch should be enough.

>> Still remaining to be fixed for making the whole suite pass:
>> issues 1781, 1782, 1759 (all three related to "make test") and 956 (links in
>> the end).
> 
> The r3d demuxer fix looks correct to me (so does mtv), maybe just commit?

Committed, for the same rationale.

-Vitor



More information about the ffmpeg-devel mailing list