Summary: | Structure property table deleted offset maps are being leaked. | ||||||
---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Gavin Barraclough <barraclough> | ||||
Component: | JavaScriptCore | Assignee: | Gavin Barraclough <barraclough> | ||||
Status: | RESOLVED FIXED | ||||||
Severity: | Normal | CC: | ddkilzer | ||||
Priority: | P1 | Keywords: | InRadar | ||||
Version: | 528+ (Nightly build) | ||||||
Hardware: | Mac | ||||||
OS: | OS X 10.5 | ||||||
Attachments: |
|
Description
Gavin Barraclough
2009-01-20 17:30:50 PST
Created attachment 26879 [details]
The patch
Comment on attachment 26879 [details]
The patch
r-, i think PropertyMapHashTable::deletedOffsets and m_propertyTable should both just be OwnPtrs so we don't need to explicitly free them
Comment on attachment 26879 [details]
The patch
r=me, i take that back, i see why they're not now
Sending JavaScriptCore/ChangeLog Sending JavaScriptCore/runtime/Structure.cpp Transmitting file data .. Committed revision 40076. Erm, r=me from bdash in channel, r+ on bug by Oliver... so, reviewed by both! |