Reviewed by Mark Rowe.
WebProcess crashes in release for simple layout tests
https://bugs.webkit.org/show_bug.cgi?id=41575
* WebProcess/InjectedBundle/API/c/WKBundlePage.cpp:
(WKBundlePageCopyRenderTreeExternalRepresentation): ref external representation StringImpl*
before taking it out of its parent String, to avoid returning freed memory.
git-svn-id: http://svn.webkit.org/repository/webkit/trunk@62461
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
+2010-07-04 Maciej Stachowiak <mjs@apple.com>
+
+ Reviewed by Mark Rowe.
+
+ WebProcess crashes in release for simple layout tests
+ https://bugs.webkit.org/show_bug.cgi?id=41575
+
+ * WebProcess/InjectedBundle/API/c/WKBundlePage.cpp:
+ (WKBundlePageCopyRenderTreeExternalRepresentation): ref external representation StringImpl*
+ before taking it out of its parent String, to avoid returning freed memory.
+
2010-07-03 Maciej Stachowiak <mjs@apple.com>
Rubber stamped by Anders Carlsson.
WKStringRef WKBundlePageCopyRenderTreeExternalRepresentation(WKBundlePageRef pageRef)
{
- WebCore::StringImpl* string = toWK(pageRef)->renderTreeExternalRepresentation().impl();
- string->ref();
- return toRef(string);
+ WebCore::String string = toWK(pageRef)->renderTreeExternalRepresentation();
+ string.impl()->ref();
+ return toRef(string.impl());
}