From 958a2612a550c8433537a73c253c9951f7015049 Mon Sep 17 00:00:00 2001
From: Dan Winship <danw@gnome.org>
Date: Wed, 26 Mar 2014 12:08:34 -0400
Subject: [PATCH] bluetooth: fix ModemManager1 code
The ModemManager1 code was assuming that the modem object would not be
created until after the Bluez Connect call returned, but that's only
true if this is the first time you tried to use it in the session. Fix
it.
NOTE: The MM 0.7 code is presumably also broken in the same way, but
is not fixed by the patch.
---
src/gnome-bluetooth/nma-bt-device.c | 94 ++++++++++++++++++++-----------------
1 file changed, 52 insertions(+), 42 deletions(-)
diff --git a/src/gnome-bluetooth/nma-bt-device.c b/src/gnome-bluetooth/nma-bt-device.c
index 8426242..3a33dce 100644
--- a/src/gnome-bluetooth/nma-bt-device.c
+++ b/src/gnome-bluetooth/nma-bt-device.c
@@ -641,10 +641,9 @@ modem_removed (DBusGProxy *proxy, const char *path, gpointer user_data)
#if WITH_MODEM_MANAGER_1
-static void
-modem_object_added (MMManager *modem_manager,
- MMObject *modem_object,
- NmaBtDevice *self)
+static gboolean
+check_modem (NmaBtDevice *self,
+ MMObject *modem_object)
{
NmaBtDevicePrivate *priv = NMA_BT_DEVICE_GET_PRIVATE (self);
NMDeviceType devtype = NM_DEVICE_TYPE_UNKNOWN;
@@ -659,11 +658,11 @@ modem_object_added (MMManager *modem_manager,
/* Ensure we have the 'Modem' interface at least */
modem_iface = mm_object_peek_modem (modem_object);
- g_return_if_fail (modem_iface != NULL);
+ g_return_val_if_fail (modem_iface != NULL, FALSE);
/* Get modem's primary port */
primary_port = mm_modem_get_primary_port (modem_iface);
- g_return_if_fail (primary_port != NULL);
+ g_return_val_if_fail (primary_port != NULL, FALSE);
/* Get rfcomm iface name */
iface_basename = g_path_get_basename (priv->rfcomm_iface);
@@ -672,7 +671,7 @@ modem_object_added (MMManager *modem_manager,
if (!g_str_equal (primary_port, iface_basename)) {
g_message ("%s: (%s) (%s) not the modem we're looking for (%s)",
__func__, path, primary_port, iface_basename);
- return;
+ return FALSE;
}
/* This is the modem we were waiting for, so keep on */
@@ -691,14 +690,16 @@ modem_object_added (MMManager *modem_manager,
/* Launch wizard! */
start_wizard (self, path, devtype);
+
+ return TRUE;
}
static void
-modem_object_removed (MMManager *manager,
- MMObject *modem_object,
- NmaBtDevice *self)
+modem_object_added (MMManager *modem_manager,
+ MMObject *modem_object,
+ NmaBtDevice *self)
{
- g_message ("%s: (%s) modem removed", __func__, mm_object_get_path (modem_object));
+ check_modem (self, modem_object);
}
#endif /* WITH_MODEM_MANAGER_1 */
@@ -712,6 +713,10 @@ dun_connect_cb (DBusGProxy *proxy,
NmaBtDevicePrivate *priv = NMA_BT_DEVICE_GET_PRIVATE (self);
GError *error = NULL;
char *device;
+#if WITH_MODEM_MANAGER_1
+ GList *modems, *iter;
+ gboolean matched = FALSE;
+#endif
g_message ("%s: processing Connect reply", __func__);
@@ -733,6 +738,42 @@ dun_connect_cb (DBusGProxy *proxy,
priv->rfcomm_iface = device;
g_message ("%s: new rfcomm interface '%s'", __func__, device);
+#if WITH_MODEM_MANAGER_1
+ /* ModemManager1 stuff */
+ priv->dbus_connection = g_bus_get_sync (G_BUS_TYPE_SYSTEM, NULL, &error);
+ if (!priv->dbus_connection) {
+ dun_error (self, __func__, error, _("error getting bus connection"));
+ g_error_free (error);
+ goto out;
+ }
+
+ priv->modem_manager_1 = mm_manager_new_sync (priv->dbus_connection,
+ G_DBUS_OBJECT_MANAGER_CLIENT_FLAGS_NONE,
+ NULL,
+ &error);
+ if (!priv->modem_manager_1) {
+ dun_error (self, __func__, error, "error creating modem manager");
+ g_error_free (error);
+ goto out;
+ }
+
+ modems = g_dbus_object_manager_get_objects (G_DBUS_OBJECT_MANAGER (priv->modem_manager_1));
+ for (iter = modems; iter; iter = iter->next) {
+ if (check_modem (self, iter->data)) {
+ matched = TRUE;
+ break;
+ }
+ }
+ g_list_free_full (modems, g_object_unref);
+
+ if (!matched) {
+ g_signal_connect (priv->modem_manager_1,
+ "object-added",
+ G_CALLBACK (modem_object_added),
+ self);
+ }
+#endif
+
out:
g_message ("%s: finished", __func__);
}
@@ -801,37 +842,6 @@ dun_start (NmaBtDevice *self)
G_CALLBACK (modem_removed), self,
NULL);
-#if WITH_MODEM_MANAGER_1
- /* ModemManager1 stuff */
- {
- GError *error = NULL;
-
- priv->dbus_connection = g_bus_get_sync (G_BUS_TYPE_SYSTEM, NULL, &error);
- if (!priv->dbus_connection) {
- dun_error (self, __func__, error, _("error getting bus connection"));
- g_error_free (error);
- } else {
- priv->modem_manager_1 = mm_manager_new_sync (priv->dbus_connection,
- G_DBUS_OBJECT_MANAGER_CLIENT_FLAGS_NONE,
- NULL,
- &error);
- if (!priv->modem_manager_1) {
- dun_error (self, __func__, error, "error creating modem manager");
- g_error_free (error);
- } else {
- g_signal_connect (priv->modem_manager_1,
- "object-added",
- G_CALLBACK (modem_object_added),
- self);
- g_signal_connect (priv->modem_manager_1,
- "object-removed",
- G_CALLBACK (modem_object_removed),
- self);
- }
- }
- }
-#endif
-
/* Bluez */
priv->dun_proxy = dbus_g_proxy_new_for_name (priv->bus,
BLUEZ_SERVICE,
--
1.8.5.3