Patches item #1655057, was opened at 2007-02-08 17:19
Message generated for change (Comment added) made by sadrul
You can respond by visiting:

Please note that this message will contain a full copy of the comment thread,
including the initial issue submission, for this request,
not just the latest update.
>Category: None
Group: None
Status: Open
Resolution: None
Priority: 5
Private: No
Submitted By: PeterTang (petertang)
>Assigned to: Mark Doliner (thekingant)
Summary: Patch for gaim_status_destroy

Initial Comment:

gaim_status_destroy(GaimStatus *status)
        g_return_if_fail(status != NULL);

        if(status->type != NULL)



I think gaim_status_type_destroy is needed to release 


>Comment By: Sadrul Habib Chowdhury (sadrul)
Date: 2007-02-09 01:42

Logged In: YES 
Originator: NO

thekingant: Gaim doesn't seem to be crashing from your patch and the
attached change-in-account patch. I tried adding, deleting new accounts,
changing the status before and after each of the operation.
File Added: change-in-account


Comment By: Sadrul Habib Chowdhury (sadrul)
Date: 2007-02-08 23:27

Logged In: YES 
Originator: NO

I have just made this change and now doing a compile!

I have also changed in gaim_account_new so that account->presence is set
*after* the status-types for the account is set.

petertang: is there any reason you filed this bug under 'ncurses'?


Comment By: Mark Doliner (thekingant)
Date: 2007-02-08 22:59

Logged In: YES 
Originator: NO

Hmm, I guess you might be right, that does look like a memleak (did you
see my email reply?  You're the same guy that emailed Luke about this a few
days ago, right?)

How did you find the memleak?  Using valgrind?  Have you tested that your
changes don't cause crashes?  Especially when adding and removing accounts
and when signing online and signing offline?

I think it would be better if GaimStatus->type was a reference to one of
the GaimStatusTypes in GaimAccount->status_types rather than a copy of a
GaimStatusType.  See my attached patch to get an idea of what I mean.  I
haven't tested it (I don't have access to a machine that I can compile on
right now).
File Added: possibly_better_memleak_fix.diff


You can respond by visiting:

Using Tomcat but need to do more? Need to support web services, security?
Get stuff done quickly with pre-integrated technology to make your job easier.
Download IBM WebSphere Application Server v.1.0.1 based on Apache Geronimo
Gaim-patches mailing list

Reply via email to