add error handling

deanlee · Apr 11, 2024 · 18d7052 · 18d7052
1 parent 70b8e27
commit 18d7052
Showing 1 changed file with 11 additions and 1 deletion.
diff --git a/selfdrive/ui/qt/network/wifi_manager.cc b/selfdrive/ui/qt/network/wifi_manager.cc
@@ -99,10 +99,20 @@ void WifiManager::refreshFinished(QDBusPendingCallWatcher *watcher) {
   seenNetworks.clear();
 
   const QDBusReply<QList<QDBusObjectPath>> wather_reply = *watcher;
+  if (!wather_reply.isValid()) {
+    qCritical() << "Failed to refresh.";
+    watcher->deleteLater();
+    return;
+  }
+
   for (const QDBusObjectPath &path : wather_reply.value()) {
     QDBusReply<QVariantMap> replay = call(path.path(), NM_DBUS_INTERFACE_PROPERTIES, "GetAll", NM_DBUS_INTERFACE_ACCESS_POINT);
-    auto properties = replay.value();
+    if (!replay.isValid()) {
+      qCritical() << "Failed to retrieve properties for path:" << path.path();
+      continue;
+    }
 
+    auto properties = replay.value();
     const QByteArray ssid = properties["Ssid"].toByteArray();
     if (ssid.isEmpty()) continue;