summaryrefslogtreecommitdiffstats
path: root/src/plugins/bearer/networkmanager
diff options
context:
space:
mode:
authorAaron McCarthy <aaron.mccarthy@nokia.com>2010-12-15 15:30:43 +1000
committerAaron McCarthy <aaron.mccarthy@nokia.com>2010-12-15 15:34:15 +1000
commite75980a322ea1088e1c6bf60259b8ba853d32444 (patch)
tree164656430a4eddc39751b4015380a5c05b0782e6 /src/plugins/bearer/networkmanager
parent6c1050611c91a88cdeaf62efe581c477873db2dc (diff)
Fix possible null pointer dereference.
When parsing a new connection an access point with the same SSID may not have been previously seen. Task-number: QTBUG-15276
Diffstat (limited to 'src/plugins/bearer/networkmanager')
-rw-r--r--src/plugins/bearer/networkmanager/qnetworkmanagerengine.cpp8
1 files changed, 5 insertions, 3 deletions
diff --git a/src/plugins/bearer/networkmanager/qnetworkmanagerengine.cpp b/src/plugins/bearer/networkmanager/qnetworkmanagerengine.cpp
index 554f9b7cfd..f93b6052e9 100644
--- a/src/plugins/bearer/networkmanager/qnetworkmanagerengine.cpp
+++ b/src/plugins/bearer/networkmanager/qnetworkmanagerengine.cpp
@@ -743,9 +743,11 @@ QNetworkConfigurationPrivate *QNetworkManagerEngine::parseConnection(const QStri
QNetworkConfigurationPrivatePointer ptr =
accessPointConfigurations.take(accessPointId);
- mutex.unlock();
- emit configurationRemoved(ptr);
- mutex.lock();
+ if (ptr) {
+ mutex.unlock();
+ emit configurationRemoved(ptr);
+ mutex.lock();
+ }
}
break;
}