commit d2a78bc6bb4258bba8a0f6bc36f86f6f5ad0cecc
author Darin Petkov <petkov@chromium.org> Wed Aug 15 12:34:49 2012 +0200
committer Gerrit <chrome-bot@google.com> Thu Aug 16 12:44:04 2012 -0700
tree a6dc324ede0dc4a991303676da2a210015d34bb0
parent 4178023c4d627410777bdc259bfc7a31f2a4b8e9

shill: cellular: Avoid shill crash when there's no SIM card inserted.

BUG=chromium-os:33409
TEST=unit tests, tested on device

Change-Id: If1b0c1ab50107150d97eab659d577eba21a0ba8f
Reviewed-on: https://gerrit.chromium.org/gerrit/30401
Tested-by: Darin Petkov <petkov@chromium.org>
Reviewed-by: Gary Morain <gmorain@chromium.org>
Commit-Ready: Darin Petkov <petkov@chromium.org>

dbus_properties_proxy.cc

diff --git a/dbus_properties_proxy.cc b/dbus_properties_proxy.cc
index dfb6d17..d24f734 100644
--- a/dbus_properties_proxy.cc
+++ b/dbus_properties_proxy.cc
@@ -23,10 +23,10 @@
   try {
     return proxy_.GetAll(interface_name);
   } catch (const DBus::Error &e) {
-    LOG(FATAL) << "DBus exception: " << e.name() << ": " << e.what()
+    LOG(ERROR) << "DBus exception: " << e.name() << ": " << e.what()
                << " interface name: " << interface_name;
-    return DBusPropertiesMap();  // Make the compiler happy.
   }
+  return DBusPropertiesMap();
 }
 
 void DBusPropertiesProxy::set_properties_changed_callback(