Adium

Opened 9 years ago

Closed 9 years ago

#15047 closed patch (fixed)

AIListOutlineView is leaking when closing the Contact List.

Reported by: ROBOTIVE Owned by: robotive
Milestone: Adium 1.4.2 Component: Adium Core
Version: Severity: normal
Keywords: memory leaks AIListOutliveView Cc:
Patch Status: Accepted

Description

Summary

AIListOutliveView is leaking when closing the Contact List.

Steps to reproduce

Using Instruments > Leaks:

  1. Start Adium
  2. Close the Contact List

Expected results

No memory leaks.

Actual results

Right, memory leaks :)

Notes

AILitsOutlineView is a mess :P

There may be other leaks here, i've managed to fix those that a tracked by Instruments > Leaks.

Patch

Patches are included for both: 1.4.2hg and 1.5hg

Attachments (4)

Adium_1_4hg_leaks.patch (4.5 KB) - added by ROBOTIVE 9 years ago.
Adium_1_5hg_leaks.patch (2.2 KB) - added by ROBOTIVE 9 years ago.
Adium-1_4hg-15047-r2.patch (799 bytes) - added by Adrian Godoroja 9 years ago.
Adium-1_5hg-15047-r2.patch (801 bytes) - added by Adrian Godoroja 9 years ago.

Download all attachments as: .zip

Change History (13)

Changed 9 years ago by ROBOTIVE

Attachment: Adium_1_4hg_leaks.patch added

Changed 9 years ago by ROBOTIVE

Attachment: Adium_1_5hg_leaks.patch added

comment:1 Changed 9 years ago by Robert

Milestone: Adium 1.4.2
Patch Status: Initially IncludedNeeds Dev Review

comment:2 Changed 9 years ago by Adrian Godoroja

Owner: set to Adrian Godoroja

comment:3 Changed 9 years ago by David Munch

Type: defectpatch

comment:4 Changed 9 years ago by Robert

Patch Status: Needs Dev ReviewAccepted
Resolution: fixed
Status: newclosed

comment:5 Changed 9 years ago by Robert Vehse

(In 4be677112537) Changelog updates, small AdiumHelp fix. Refs #15047.

comment:6 Changed 9 years ago by Robert

Summary: AIListOutliveView is leaking when closing the Contact List.AIListOutlineView is leaking when closing the Contact List.

Changed 9 years ago by Adrian Godoroja

Attachment: Adium-1_4hg-15047-r2.patch added

Changed 9 years ago by Adrian Godoroja

Attachment: Adium-1_5hg-15047-r2.patch added

comment:7 Changed 9 years ago by Adrian Godoroja

Patch Status: AcceptedNeeds Dev Review
Resolution: fixed
Status: closednew

comment:8 Changed 9 years ago by Adrian Godoroja

Fixed two more leaks!

comment:9 Changed 9 years ago by Robert

Patch Status: Needs Dev ReviewAccepted
Resolution: fixed
Status: newclosed
Note: See TracTickets for help on using tickets.