bluez: Fix some memory leaks on device discovery.

QDBusPendingCallWatcher will leak if it isn't deleted.

Change-Id: I5571dfbc087f54308193dc89127e974e6d2e9275
Reviewed-by: Alex Blasche <alexander.blasche@theqtcompany.com>

1 files changed, 7 insertions, 2 deletions

diff --git a/src/bluetooth/qbluetoothservicediscoveryagent_bluez.cpp b/src/bluetooth/qbluetoothservicediscoveryagent_bluez.cpp
index f66603a9..d186eafc 100644
--- a/src/bluetooth/qbluetoothservicediscoveryagent_bluez.cpp
+++ b/src/bluetooth/qbluetoothservicediscoveryagent_bluez.cpp
@@ -354,8 +354,10 @@ void QBluetoothServiceDiscoveryAgentPrivate::stop()
 
 void QBluetoothServiceDiscoveryAgentPrivate::_q_createdDevice(QDBusPendingCallWatcher *watcher)
 {
-    if (!adapter)
+    if (!adapter) {
+        watcher->deleteLater();
         return;
+    }
 
     Q_Q(QBluetoothServiceDiscoveryAgent);
 
@@ -364,6 +366,7 @@ void QBluetoothServiceDiscoveryAgentPrivate::_q_createdDevice(QDBusPendingCallWa
     qCDebug(QT_BT_BLUEZ) << Q_FUNC_INFO << "created" << address.toString();
 
     QDBusPendingReply<QDBusObjectPath> deviceObjectPath = *watcher;
+    watcher->deleteLater();
     if (deviceObjectPath.isError()) {
         if (deviceObjectPath.error().name() != QStringLiteral("org.bluez.Error.AlreadyExists")) {
             delete adapter;
@@ -488,8 +491,10 @@ void QBluetoothServiceDiscoveryAgentPrivate::_q_createdDevice(QDBusPendingCallWa
 // Bluez 4
 void QBluetoothServiceDiscoveryAgentPrivate::_q_discoveredServices(QDBusPendingCallWatcher *watcher)
 {
-    if (!device)
+    if (!device) {
+        watcher->deleteLater();
         return;
+    }
 
     qCDebug(QT_BT_BLUEZ) << Q_FUNC_INFO;
     Q_Q(QBluetoothServiceDiscoveryAgent);