In method_new() the method is append to inlist of methods in interface.

On Mon, Feb 18, 2013 at 1:21 PM, Daniel Willmann <d.willm...@samsung.com>wrote:

> 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
> > <d.willm...@samsung.com>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
enlightenment-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/enlightenment-devel

Reply via email to