Bug 43229

Summary: REGRESSION: False-positive memory leak within JSArray::JSArray
Product: WebKit Reporter: Mark Rowe (bdash) <mrowe>
Component: JavaScriptCoreAssignee: Michael Saboff <msaboff>
Status: RESOLVED FIXED    
Severity: Normal CC: abarth, barraclough, eric, ggaren, msaboff, webkit.review.bot
Priority: P1 Keywords: InRadar, Regression
Version: 528+ (Nightly build)   
Hardware: PC   
OS: OS X 10.5   
URL: http://build.webkit.org/results/SnowLeopard%20Intel%20Leaks/r64324%20(9455)/
Attachments:
Description Flags
Patch to save alloc'ed pointer in ArrayAStorage structure.
barraclough: review-
Updated patch without redundant void* casts. barraclough: review+

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