Re: Leak in tny-camel-mime-part



All approved, great work. Please commit

On Mon, 2007-12-17 at 20:03 +0100, Alberto Garcia wrote:
> O Luns 17/12/2007 ás 19:53, Philip Van Hoof dicía...
> 
> > > > Looks like there's a leak here, as
> > > > camel_medium_set_content_object() adds a reference
> > > > (camel_medium_get_content_object() doesn't, by the way)
> > > 
> > > And it seems that there's another one in
> > > tny_camel_mime_part_construct_from_stream_default(), it lacks a
> > > camel_object_unref (wrapper) at the end of the function.
> > 
> > Okay, please provide a patch for the second one and send it to this
> > mailing list.
> 
> I was looking for another one, and I finally found it, this one was
> probably the biggest one.
> 
> I'm attaching the full patch.
> 
> _______________________________________________
> tinymail-devel-list mailing list
> tinymail-devel-list gnome org
> http://mail.gnome.org/mailman/listinfo/tinymail-devel-list
-- 
Philip Van Hoof, freelance software developer
home: me at pvanhoof dot be 
gnome: pvanhoof at gnome dot org 
http://pvanhoof.be/blog
http://codeminded.be






[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]