commit e98408012ac882b9bd6da43c69c05ecc05eafa76
author Nathan Williams <njw@chromium.org> Wed Apr 18 18:47:40 2012 -0400
committer Gerrit <chrome-bot@google.com> Thu Apr 19 07:45:12 2012 -0700
tree a6408c00a0467310f87b79da325985836be5b8bd
parent 633f639d8f433fd939949b12bd9f695162533728

shill: SIM property is an object path, not a string

Add DBusProperties::GetObjectPath() helper to accomodate this.

BUG=None
TEST=Run on a system with modemmanager-next and a GSM modem; see if it crashes.

Change-Id: I55df239b5b69e260fb1ad1d7983154ee717727b2
Reviewed-on: https://gerrit.chromium.org/gerrit/20516
Reviewed-by: Jason Glasgow <jglasgow@chromium.org>
Commit-Ready: Nathan J. Williams <njw@chromium.org>
Tested-by: Nathan J. Williams <njw@chromium.org>

cellular_capability_universal.cc

diff --git a/cellular_capability_universal.cc b/cellular_capability_universal.cc
index c9b8547..cd38d34 100644
--- a/cellular_capability_universal.cc
+++ b/cellular_capability_universal.cc
@@ -853,8 +853,8 @@
     const DBusPropertiesMap &properties,
     const vector<string> &/* invalidated_properties */) {
   string value;
-  if (DBusProperties::GetString(properties,
-                                MM_MODEM_PROPERTY_SIM, &value))
+  if (DBusProperties::GetObjectPath(properties,
+                                    MM_MODEM_PROPERTY_SIM, &value))
     OnSimPathChanged(value);
 
   uint32 access_technologies = MM_MODEM_ACCESS_TECHNOLOGY_UNKNOWN;