|
From: Mark D. <ma...@ki...> - 2007-02-07 16:42:21
|
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 |