pidgin-devel

[Gaim-devel] Fwd: memory leak of Gaim

[Luke S. <lsc...@us...>]

----- Forwarded message from Yang Tom <cb...@ho...> -----

Date: Fri, 02 Feb 2007 10:46:09 +0800
From: Yang Tom <cb...@ho...>
To: se...@ho...
Cc: lsc...@us...
Subject: memory leak of Gaim

there are memory leak in=20
		g_list_foreach(presence->statuses,=20
		(GFunc)gaim_status_destroy, NULL);
because of the types allocated by msn_status_types can't be released=20

So I think some code should be added in function gaim_status_destroy to=20
release=20
status->type member

I have sent you a mail with name ty5...@ya...

I want to be as buddy of you on MSN

Best regard=20

Peter

_________________________________________________________________
=E4=B8=8E=E8=81=94=E6=9C=BA=E7=9A=84=E6=9C=8B=E5=8F=8B=E8=BF=9B=E8=A1=8C=E4=
=BA=A4=E6=B5=81=EF=BC=8C=E8=AF=B7=E4=BD=BF=E7=94=A8 MSN Messenger: =20
http://messenger.msn.com/cn =20

----- End forwarded message -----

Re: [Gaim-devel] Fwd: memory leak of Gaim

[Mark D. <ma...@ki...>]

On Wed, 7 Feb 2007 11:28:40 -0500, Luke Schierer wrote
> ----- Forwarded message from Yang Tom <cb...@ho...> -----
> there are memory leak in 
> 		g_list_foreach(presence->statuses, 
> 		(GFunc)gaim_status_destroy, NULL);
> because of the types allocated by msn_status_types can't be released
> 
> So I think some code should be added in function gaim_status_destroy 
> to release status->type member
> ----- End forwarded message -----

I don't think this is a memory leak.  I think GaimStatus->type is a reference
to a GaimStatusType, but it's not an actual copy of the GaimStatusType.  Each
GaimAccount has a list of GaimStatusTypes (account->status_types), and I think
GaimStatus->type is a reference to one of the items in this list.

-Mark