Add null httpReply checks to QHttpThreadDelegate
authorShane Kearns <ext-shane.2.kearns@nokia.com>
Thu, 28 Jun 2012 13:52:45 +0000 (14:52 +0100)
committerQt by Nokia <qt-info@nokia.com>
Mon, 2 Jul 2012 23:09:25 +0000 (01:09 +0200)
commit41064f851591d9437baeda502b6e2504fee8f213
treefcf228faddbf5e2d118fb8f9ce88504c952aeac5
parent48925d98f3970f3b0b8d66839e6c2046e18fa964
Add null httpReply checks to QHttpThreadDelegate

If a request is aborted while under load, the abort signal can be
queued in front of a signal emitted from the httpReply.
The abort slot is deleting the httpReply and setting it to null.
So when the queued slot is processed the httpReply is null and
caused an MMU fault.

Removed qWarning from existing null checks, as these are expected
if abort is called with precise timing so that it races with the
reply finishing on the socket.

Task-number: QTBUG-26245
Change-Id: I0a7e0223fda1bc01d117fe8a993c7f6e43fd72ff
Reviewed-by: Jonas Gastal <jgastal@profusion.mobi>
Reviewed-by: Richard J. Moore <rich@kde.org>
src/network/access/qhttpthreaddelegate.cpp