Bug 15800

Summary: REGRESSION (r27299-r27305): RegExp leaks on Tiger
Product: WebKit Reporter: mitz
Component: JavaScriptCoreAssignee: Alexey Proskuryakov <ap>
Severity: Normal CC: ap
Priority: P1 Keywords: Regression
Version: 523.x (Safari 3)   
Hardware: Mac   
OS: OS X 10.4   
URL: http://build.webkit.org/results/trunk-mac-intel-debug/1950/DumpRenderTree2-leaks.txt
Description Flags
proposed fix darin: review+

Description mitz 2007-11-02 11:03:26 PDT
The build bot is seeing leaks starting with build 1950 <http://build.webkit.org/builders/trunk-mac-intel-debug/builds/1950>. I cannot reproduce the leaks on my Leopard system. Given the backtraces I suspect <http://trac.webkit.org/projects/webkit/changeset/27303>.
Comment 1 Alexey Proskuryakov 2007-11-03 01:15:40 PDT
Created attachment 17021 [details]
proposed fix
Comment 2 Darin Adler 2007-11-03 01:20:38 PDT
Comment on attachment 17021 [details]
proposed fix

Looks good.

Why did my code change cause the regression? Could we use OwnPtr here?

Comment 3 Alexey Proskuryakov 2007-11-03 01:51:42 PDT
(In reply to comment #2)

> Why did my code change cause the regression?

  It was my change that added RegExp.compile - I wrongly assumed setRegExp deleted the old value.

> Could we use OwnPtr here?

  Oops, forgot about it somehow. Changed to use OwnPtr.

> r=me

Committed revision 27393.