Maksym, I had to revert 2 of these 3 patches because they introduced a
dire regression:

https://sourceforge.net/p/mlt/bugs/181/

On Thu, Jul 19, 2012 at 5:10 AM, Maksym Veremeyenko <[email protected]> wrote:
> hi,
>
> attached patch fixes first three leaks in producer_avformat.
>
> unfixed leak i found comes from mlt_cache
>
> ==11279== 10,612,830 (880 direct, 10,611,950 indirect) bytes in 10 blocks
> are definitely lost in loss record 50 of 50
> ==11279==    at 0x4A05BB4: calloc (vg_replace_malloc.c:467)
> ==11279==    by 0x4C34BA7: mlt_frame_init (mlt_frame.c:44)
> ==11279==    by 0x4C35EE9: mlt_frame_clone (mlt_frame.c:994)
> ==11279==    by 0x4C4AFAB: mlt_cache_put_frame (mlt_cache.c:552)
> ==11279==    by 0xC0BC1C5: producer_get_image (producer_avformat.c:1711)
> ==11279==    by 0x4C35099: mlt_frame_get_image (mlt_frame.c:452)
> ==11279==    by 0xB683478: filter_get_image (filter_crop.c:77)
> ==11279==    by 0x4C35099: mlt_frame_get_image (mlt_frame.c:452)
> ==11279==    by 0x101E2AB2: filter_get_image (filter_deinterlace.c:271)
> ==11279==    by 0x4C35099: mlt_frame_get_image (mlt_frame.c:452)
> ==11279==    by 0xB688BFA: filter_get_image (filter_rescale.c:217)
> ==11279==    by 0x4C35099: mlt_frame_get_image (mlt_frame.c:452)
> ==11279==
>
> that seems happens because of frames caching is hack to origin cache
> functionality and mlt_cache_close do not destroy cached frames.
>
> would be a good idea to create a local cache in producer_avformat?
>

-- 
+-DRD-+

------------------------------------------------------------------------------
LogMeIn Rescue: Anywhere, Anytime Remote support for IT. Free Trial
Remotely access PCs and mobile devices and provide instant support
Improve your efficiency, and focus on delivering more value-add services
Discover what IT Professionals Know. Rescue delivers
http://p.sf.net/sfu/logmein_12329d2d
_______________________________________________
Mlt-devel mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/mlt-devel

Reply via email to