On 18/02/13 14:35, Jose Souza wrote:
> Hi Daniel
> 
> I don't see any leaks.
> method->cb_name = strdup("NULL"); is because the method have no reply
> annotation and this is only know after "</method>"
> The DBus_Method and all strings of they will be freed when
> object_free()->inteface_free()->method_free() is called.

Method is set to NULL at the end of method_close(), but I guess there
must be some other reference to that pointer that still exists somewhere?

Otherwise you're doing

method->cb_name = strdup("NULL");
method = NULL;

> On Mon, Feb 18, 2013 at 10:53 AM, Daniel Willmann
> <[email protected]>wrote:
> 
>> Hello,
>>
>> could someone with knowledge please take a look at method_close() in
>> src/bin/edbus/parser.c
>>
>> It seems like the method->cb_name = strdup("NULL"); before free(method);
>> leaks memory. I know too little about where/how the global var method is
>> being used to decide here.


Daniel

------------------------------------------------------------------------------
The Go Parallel Website, sponsored by Intel - in partnership with Geeknet, 
is your hub for all things parallel software development, from weekly thought 
leadership blogs to news, videos, case studies, tutorials, tech docs, 
whitepapers, evaluation guides, and opinion stories. Check out the most 
recent posts - join the conversation now. http://goparallel.sourceforge.net/
_______________________________________________
enlightenment-devel mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/enlightenment-devel

Reply via email to