hfp_hf_bluez5: Fix crash on re-pairing a Device

It may happen that a Device object is unpaired an paired again
without being removed from DBus. This in turn triggers second
modem object to be created, but not fully initialized.
If this modem object is used, oFono will crash.

diff --git a/plugins/hfp_hf_bluez5.c b/plugins/hfp_hf_bluez5.c
index 5ee70c5..713c789 100644 (file)
--- a/plugins/hfp_hf_bluez5.c
+++ b/plugins/hfp_hf_bluez5.c
@@ -738,8 +738,17 @@ static void modem_register_from_proxy(GDBusProxy *proxy, const char *path)
                return;
 
        dbus_message_iter_get_basic(&iter, &paired);
-       if (paired == FALSE)
+
+       if (paired == FALSE) {
+               modem = ofono_modem_find(device_path_compare, (void *) path);
+
+               if (modem != NULL) {
+                       ofono_modem_remove(modem);
+                       g_dbus_proxy_set_removed_watch(proxy, NULL, NULL);
+                       g_dbus_proxy_set_property_watch(proxy, NULL, NULL);
+               }
                return;
+       }
 
        if (g_dbus_proxy_get_property(proxy, "UUIDs", &iter) == FALSE)
                return;