Closed Bug 369423 Opened 17 years ago Closed 17 years ago

XULSortServiceImpl leaks array of contentSortInfo

Categories

(Core :: DOM: Core & HTML, defect)

defect
Not set
normal

Tracking

()

RESOLVED FIXED

People

(Reporter: peterv, Assigned: peterv)

References

Details

(Keywords: memory-leak)

Attachments

(1 file)

Attached patch v1Splinter Review
ULSortServiceImpl puts a number of contentSortInfo's in a nsTArray<contentSortInfo*> and then never deletes them. Each of the contentSortInfo structs holds three nsCOMPtr's. Simply replacing nsTArray<contentSortInfo*> with nsTArray<contentSortInfo> fixes the leak.
Attachment #254098 - Flags: superreview?(jonas)
Attachment #254098 - Flags: review?(enndeakin)
Attachment #254098 - Flags: superreview?(jonas) → superreview+
Attachment #254098 - Flags: review?(enndeakin) → review+
Status: NEW → RESOLVED
Closed: 17 years ago
Resolution: --- → FIXED
Keywords: mlk
Component: DOM → DOM: Core & HTML
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: