Bug 28029

Summary: Fix DOM Storage memory leak
Product: WebKit Reporter: Jeremy Orlow <jorlow>
Component: New BugsAssignee: Nobody <webkit-unassigned>
Status: RESOLVED FIXED    
Severity: Normal CC: fishd
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: Other   
OS: OS X 10.5   
Attachments:
Description Flags
Patch v1
none
Patch v1 fishd: review+

Jeremy Orlow
Reported 2009-08-05 15:16:00 PDT
Created attachment 34185 [details] Patch v1 Forgot to use adoptRef in one place. This caused the = operator to increment the ref count when it should have just left it as is. The result of this was that StorageAreaImpl's were never being freed.
Attachments
Patch v1 (1.17 KB, patch)
2009-08-05 15:16 PDT, Jeremy Orlow 		no flags
DetailsFormatted DiffDiff
Patch v1 (1.36 KB, patch)
2009-08-05 15:18 PDT, Jeremy Orlow 		fishd: review+
DetailsFormatted DiffDiff
Show Obsolete (1) View All Add attachment proposed patch, testcase, etc.
Jeremy Orlow
Comment 1 2009-08-05 15:18:11 PDT
Created attachment 34187 [details] Patch v1
Jeremy Orlow
Comment 2 2009-08-05 15:48:52 PDT
Sending WebCore/ChangeLog Sending WebCore/storage/StorageNamespaceImpl.cpp Transmitting file data .. Committed revision 46818. http://trac.webkit.org/changeset/46818
Note You need to log in before you can comment on or make changes to this bug.