diff options
author | Jocelyn Turcotte <jocelyn.turcotte@digia.com> | 2013-05-14 08:31:16 +0000 |
---|---|---|
committer | The Qt Project <gerrit-noreply@qt-project.org> | 2013-06-06 13:02:59 +0200 |
commit | b3923742ae3271ec1e273a5ffbc5555f3ee089ae (patch) | |
tree | dfaaba2417f0e587c316ebda83f5969dd978a5ae /Source/JavaScriptCore/runtime/JSONObject.cpp | |
parent | 0c06c599cfa5a3235922494992712860c34dfcd3 (diff) | |
download | qtwebkit-b3923742ae3271ec1e273a5ffbc5555f3ee089ae.tar.gz |
[Qt] Fix a crash under ~PingLoader when the QNAM on the page has been destroyed.
https://bugs.webkit.org/show_bug.cgi?id=116035
Reviewed by Simon Hausmann.
Source/WebCore:
Reproduced with arora which does destroy the QNetworkAccessManager in some situations.
The problem is that PingLoader can still be pending meanwhile, holding a ResourceHandle
with a dangling pointer to a QNetworkReply destroyed with the QNetworkAccessManager.
* platform/network/qt/QNetworkReplyHandler.cpp:
(WebCore::QNetworkReplyWrapper::QNetworkReplyWrapper):
Set the parent to 0 like we did before the introduction of QNetworkReplyWrapper.
(WebCore::QNetworkReplyWrapper::release):
Source/WebKit/qt:
* tests/qwebpage/tst_qwebpage.cpp:
(tst_QWebPage::networkReplyParentChanged):
Change-Id: Id53db216a6252f8471d6fa97093312843213e48d
git-svn-id: http://svn.webkit.org/repository/webkit/trunk@150057 268f45cc-cd09-0410-ab3c-d52691b4dbfc
Reviewed-by: Allan Sandfeld Jensen <allan.jensen@digia.com>
Diffstat (limited to 'Source/JavaScriptCore/runtime/JSONObject.cpp')
0 files changed, 0 insertions, 0 deletions