From bc165ef40f35be5695d2fc575e5667b4dd33eeb9 Mon Sep 17 00:00:00 2001 From: NeilClack Date: Wed, 12 Aug 2026 14:38:22 -0500 Subject: [PATCH] fix(network): read AccessPoints instead of AllAccessPoints for scan list AllAccessPoints is not exposed by this NetworkManager ('No such property'), so getAccessPoints() hit the error path and returned an empty scan list with no log output. AccessPoints is the current scan results and is universally supported. Also log the property-read failure instead of failing silently. --- src/network/nm_device.cpp | 7 +++++-- 1 file changed, 5 insertions(+), 2 deletions(-) diff --git a/src/network/nm_device.cpp b/src/network/nm_device.cpp index 47e0930..73b6aaa 100644 --- a/src/network/nm_device.cpp +++ b/src/network/nm_device.cpp @@ -34,7 +34,7 @@ AccessPointInfo parseAccessPoint( ap.wpa_flags = wpa_flags; ap.rsn_flags = rsn_flags; ap.security = detectSecurityType(flags, wpa_flags, rsn_flags); - + return ap; } @@ -211,13 +211,16 @@ std::vector WifiDevice::getAccessPoints() const { "org.freedesktop.NetworkManager", impl_->object_path.c_str(), "org.freedesktop.NetworkManager.Device.Wireless", - "AllAccessPoints", + "AccessPoints", &error, &msg, "ao" ); if (r < 0) { + radiantwave::logger::Logger::instance().warning("WiFi", + "Failed to read AccessPoints on device " + impl_->object_path + ": " + + (error.message ? error.message : strerror(-r))); sd_bus_error_free(&error); return result; }