shill: cellular: No need to crash if ModemManager's EnumerateDevices call fails.
BUG=chromium-os:33652
TEST=see bug report
Change-Id: I92e8980eb714c8d1b660fe166b540dfa61a4c422
Reviewed-on: https://gerrit.chromium.org/gerrit/33690
Tested-by: Darin Petkov <petkov@chromium.org>
Reviewed-by: Ben Chan <benchan@chromium.org>
Commit-Ready: Darin Petkov <petkov@chromium.org>
diff --git a/modem_manager_proxy.cc b/modem_manager_proxy.cc
index 007da6d..b89b3e6 100644
--- a/modem_manager_proxy.cc
+++ b/modem_manager_proxy.cc
@@ -1,4 +1,4 @@
-// Copyright (c) 2011 The Chromium OS Authors. All rights reserved.
+// Copyright (c) 2012 The Chromium OS Authors. All rights reserved.
// Use of this source code is governed by a BSD-style license that can be
// found in the LICENSE file.
@@ -25,9 +25,9 @@
try {
return proxy_.EnumerateDevices();
} catch (const DBus::Error &e) {
- LOG(FATAL) << "DBus exception: " << e.name() << ": " << e.what();
- return vector<DBus::Path>(); // Make the compiler happy.
+ LOG(ERROR) << "DBus exception: " << e.name() << ": " << e.what();
}
+ return vector<DBus::Path>();
}
ModemManagerProxy::Proxy::Proxy(DBus::Connection *connection,