From 99d2e0f8616c69bb685ab51d4766095e7e3f27eb Mon Sep 17 00:00:00 2001
From: Edward Welbourne <edward.welbourne@qt.io>
Date: Wed, 24 Aug 2016 14:56:24 +0200
Subject: [PATCH] tst_qnetworkreply: condition dereferencing on non-null

The (event) loop member spends most of its time NULL, so make sure
slots that dereference it can't trip up if called asynchronously when
it is.

Change-Id: If634df0ecf9650b52621bdb5a3e9f151abbc18fc
Reviewed-by: Timur Pocheptsov <timur.pocheptsov@theqtcompany.com>
---
 .../auto/network/access/qnetworkreply/tst_qnetworkreply.cpp | 6 ++++--
 1 file changed, 4 insertions(+), 2 deletions(-)

diff --git a/tests/auto/network/access/qnetworkreply/tst_qnetworkreply.cpp b/tests/auto/network/access/qnetworkreply/tst_qnetworkreply.cpp
index 29b3a3e026..6fad7cbc0b 100644
--- a/tests/auto/network/access/qnetworkreply/tst_qnetworkreply.cpp
+++ b/tests/auto/network/access/qnetworkreply/tst_qnetworkreply.cpp
@@ -1456,12 +1456,14 @@ int tst_QNetworkReply::waitForFinish(QNetworkReplyPtr &reply)
 
 void tst_QNetworkReply::finished()
 {
-    loop->exit(returnCode = Success);
+    if (loop)
+        loop->exit(returnCode = Success);
 }
 
 void tst_QNetworkReply::gotError()
 {
-    loop->exit(returnCode = Failure);
+    if (loop)
+        loop->exit(returnCode = Failure);
     disconnect(QObject::sender(), SIGNAL(finished()), this, 0);
 }