Bug 43229 - REGRESSION: False-positive memory leak within JSArray::JSArray
Summary: REGRESSION: False-positive memory leak within JSArray::JSArray
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: JavaScriptCore (show other bugs)
Version: 528+ (Nightly build)
Hardware: PC OS X 10.5
: P1 Normal
Assignee: Michael Saboff
URL: http://build.webkit.org/results/SnowL...
Keywords: InRadar, Regression
Depends on:
Blocks:
 
Reported: 2010-07-29 18:46 PDT by Mark Rowe (bdash)
Modified: 2010-08-03 14:48 PDT (History)
6 users (show)

See Also:

Attachments
Patch to save alloc'ed pointer in ArrayAStorage structure. (6.69 KB, patch)
2010-08-02 10:58 PDT, Michael Saboff 		barraclough: review-
Details | Formatted Diff | Diff
Updated patch without redundant void* casts. (6.63 KB, patch)
2010-08-02 13:57 PDT, Michael Saboff 		barraclough: review+
Details | Formatted Diff | Diff
Show Obsolete (1) View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Mark Rowe (bdash) 2010-07-29 18:46:45 PDT 
The buildbot is showing many memory leaks within JSArray.  See the numerous DumpRenderTreeFoo-leaks.txt files at <http://build.webkit.org/results/SnowLeopard%20Intel%20Leaks/r64324%20(9455)/>, such as <http://build.webkit.org/results/SnowLeopard%20Intel%20Leaks/r64324%20(9455)/DumpRenderTree14-leaks.txt>.
Comment 1 Mark Rowe (bdash) 2010-07-29 18:48:13 PDT 
<rdar://problem/8254202>
Comment 2 Michael Saboff 2010-08-02 10:58:50 PDT 
Created attachment 63237 [details]
Patch to save alloc'ed pointer in ArrayAStorage structure.
Comment 3 Gavin Barraclough 2010-08-02 13:46:19 PDT 
Comment on attachment 63237 [details]
Patch to save alloc'ed pointer in ArrayAStorage structure.

Please remove the redundant static_cast<void*>()s
Comment 4 Michael Saboff 2010-08-02 13:57:21 PDT 
Created attachment 63257 [details]
Updated patch without redundant void* casts.
Comment 5 Gavin Barraclough 2010-08-03 14:29:43 PDT 
Transmitting file data ....
Committed revision 64588.
Comment 6 WebKit Review Bot 2010-08-03 14:48:59 PDT 
http://trac.webkit.org/changeset/64588 might have broken GTK Linux 64-bit Debug