[sylpheed:31979] Re: [PATCH] Sylpheed 2.4.7 memory leak in LDAP
Hiroyuki Yamamoto
hiro-y at kcn.ne.jp
Tue Dec 18 11:10:10 JST 2007
Hello,
On Mon, 17 Dec 2007 15:56:20 +0100
"Michael Schwendt" <mschwendt at gmail.com> wrote:
> On 17/12/2007, Hiroyuki Yamamoto wrote:
> >
> > > In syldap.c, function syldap_search(), in the "/* Process all
> > > attributes */" loop, why is only the last "attribute" pointer
> > > freed with ldap_memfree() after the loop is done? As I understand
> > > it, both ldap_first_attribute() and ldap_next_attribute() return
> > > a dynamically allocated array of char every time (or NULL), which
> > > ought to be freed after each iteration of the loop. IMO:
> >
> > You are correct, and there were four more leaks in syldap.c :)
> > Fixed in svn trunk.
>
> Good!
>
> The following warning remains, though, as the second patch was not
> applied completely:
>
> (sylpheed:19612): GLib-CRITICAL **: g_source_remove: assertion `tag
> > 0' failed
It should be fixed correctly now.
--
Hiroyuki Yamamoto <hiro-y at kcn.ne.jp>
More information about the Sylpheed
mailing list