shill: WiFi: Do not crash on RemoveNetwork failure
The failure of this call should not cause shill to crash when
called from DisconnectFrom(). It can happen during system
shutdown when shill and wpa_supplicant race to shut down.
wpa_supplicant's DBus endpoint could disappear just after we
enter the shill exit handler.
BUG=chromium-os:33658
TEST=Rerun unit tests. MatFunc 000
Change-Id: I21d599cdc495e0d1246d450306847c0a8be752c7
Reviewed-on: https://gerrit.chromium.org/gerrit/33231
Reviewed-by: mukesh agrawal <quiche@chromium.org>
Commit-Ready: Paul Stewart <pstew@chromium.org>
Tested-by: Paul Stewart <pstew@chromium.org>
diff --git a/supplicant_interface_proxy.cc b/supplicant_interface_proxy.cc
index 807ccf0..ad581a2 100644
--- a/supplicant_interface_proxy.cc
+++ b/supplicant_interface_proxy.cc
@@ -92,7 +92,8 @@
try {
return proxy_.RemoveNetwork(network);
} catch (const DBus::Error &e) {
- LOG(FATAL) << "DBus exception: " << e.name() << ": " << e.what();
+ LOG(ERROR) << "DBus exception: " << e.name() << ": " << e.what();
+ throw; // Re-throw the exception.
}
}