Closed Bug 488385 Opened 17 years ago Closed 17 years ago

Memory leak in QCache.cpp

Categories

(Tamarin Graveyard :: Virtual Machine, defect)

defect
Not set
normal

Tracking

(Not tracked)

VERIFIED FIXED

People

(Reporter: stejohns, Assigned: stejohns)

Details

Attachments

(2 files)

Attached patch PatchSplinter Review
When evicted a cached item, we neglect to set its 'next' field to null. If the evicted item is held alive by virtue of being an ancestor of a cached item, it's bogus 'next' link can hold an arbitrary chain of QCachedItems alive.
Attachment #372727 - Flags: review?(treilly)
Attached patch Patch #2Splinter Review
A few changes that helped track down this bug: improvements to the accuracy of AVMPLUS_MEMTRACK mode.
Attachment #372743 - Flags: review?(treilly)
Attachment #372727 - Flags: review?(treilly) → review+
Attachment #372743 - Flags: review?(treilly) → review+
pushed to redux as changeset: 1753:d36fc7966690
Status: NEW → RESOLVED
Closed: 17 years ago
Resolution: --- → FIXED
Resolved fixed engineering / work item that has been pushed. Setting status to verified.
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: