[FFmpeg-devel] [PATCH] fix imlib2 memory leaks

Benoit Fouet benoit.fouet
Mon Jun 18 14:00:36 CEST 2007


Michael Niedermayer wrote:
> Hi
>
> On Mon, Jun 18, 2007 at 10:22:42AM +0800, Limin Wang wrote:
>   
>> Index: vhook/imlib2.c
>> ===================================================================
>> --- vhook/imlib2.c	(revision 9358)
>> +++ vhook/imlib2.c	(working copy)
>> @@ -174,11 +174,17 @@
>>              imlib_context_set_image(ci->imageOverlaid);
>>              imlib_free_image();
>>          }
>> -        ff_eval_free(ci->expr_x);
>> -        ff_eval_free(ci->expr_y);
>> -        ff_eval_free(ci->expr_R);
>> -        ff_eval_free(ci->expr_G);
>> -        ff_eval_free(ci->expr_B);
>> +        ff_eval_free(ci->eval_x);
>> +        ff_eval_free(ci->eval_y);
>> +        ff_eval_free(ci->eval_r);
>> +        ff_eval_free(ci->eval_g);
>> +        ff_eval_free(ci->eval_b);
>> +
>> +        av_free(ci->expr_x);
>> +        av_free(ci->expr_y);
>> +        av_free(ci->expr_R);
>> +        av_free(ci->expr_G);
>> +        av_free(ci->expr_B);
>>          sws_freeContext(ci->toRGB_convert_ctx);
>>          sws_freeContext(ci->fromRGB_convert_ctx);
>>          av_free(ctx);
>>     
>
> this looks ok
>
> [...]
>   
>   

Applied

Ben
-- 
Purple Labs S.A.
www.purplelabs.com




More information about the ffmpeg-devel mailing list