Merge branch 'moonshot' of ssh://moonshot.suchdamage.org:822/srv/git/libeap into...
[libeap.git] / wpa_supplicant / dbus / dbus_new.c
index 3d5a016..bdfbbac 100644 (file)
@@ -1,7 +1,7 @@
 /*
  * WPA Supplicant / dbus-based control interface
  * Copyright (c) 2006, Dan Williams <dcbw@redhat.com> and Red Hat, Inc.
- * Copyright (c) 2009, Witold Sowa <witold.sowa@gmail.com>
+ * Copyright (c) 2009-2010, Witold Sowa <witold.sowa@gmail.com>
  * Copyright (c) 2009, Jouni Malinen <j@w1.fi>
  *
  * This program is free software; you can redistribute it and/or modify
@@ -181,7 +181,7 @@ static void wpas_dbus_signal_bss(struct wpa_supplicant *wpa_s,
                        goto err;
 
                wpa_dbus_get_object_properties(iface, bss_obj_path,
-                                              WPAS_DBUS_NEW_IFACE_BSSID,
+                                              WPAS_DBUS_NEW_IFACE_BSS,
                                               &iter_dict);
 
                if (!wpa_dbus_dict_close_write(&iter, &iter_dict))
@@ -393,75 +393,6 @@ void wpas_dbus_signal_network_selected(struct wpa_supplicant *wpa_s, int id)
 }
 
 
-static void str_to_lower(char *s)
-{
-       while (*s) {
-               *s = tolower(*s);
-               s++;
-       }
-}
-
-
-/**
- * wpas_dbus_signal_state_changed - Send a state changed signal
- * @wpa_s: %wpa_supplicant network interface data
- * @new_state: new state wpa_supplicant is entering
- * @old_state: old state wpa_supplicant is leaving
- *
- * Notify listeners that wpa_supplicant has changed state
- */
-void wpas_dbus_signal_state_changed(struct wpa_supplicant *wpa_s,
-                                   enum wpa_states new_state,
-                                   enum wpa_states old_state)
-{
-       struct wpas_dbus_priv *iface;
-       DBusMessage *msg;
-       char *new_state_str, *old_state_str;
-
-       /* Do nothing if the control interface is not turned on */
-       if (wpa_s->global == NULL)
-               return;
-       iface = wpa_s->global->dbus;
-       if (iface == NULL)
-               return;
-
-       /* Only send signal if state really changed */
-       if (new_state == old_state)
-               return;
-
-       msg = dbus_message_new_signal(wpa_s->dbus_new_path,
-                                     WPAS_DBUS_NEW_IFACE_INTERFACE,
-                                     "StateChanged");
-       if (msg == NULL)
-               return;
-
-       new_state_str = os_strdup(wpa_supplicant_state_txt(new_state));
-       old_state_str = os_strdup(wpa_supplicant_state_txt(old_state));
-       if (new_state_str == NULL || old_state_str == NULL)
-               goto out;
-
-       /* make state string lowercase to fit new DBus API convention */
-       str_to_lower(new_state_str);
-       str_to_lower(old_state_str);
-
-       if (!dbus_message_append_args(msg,
-                                     DBUS_TYPE_STRING, &new_state_str,
-                                     DBUS_TYPE_STRING, &old_state_str,
-                                     DBUS_TYPE_INVALID)) {
-               wpa_printf(MSG_ERROR, "dbus: Failed to construct state change "
-                          "signal");
-               goto out;
-       }
-
-       dbus_connection_send(iface->con, msg, NULL);
-
-out:
-       dbus_message_unref(msg);
-       os_free(new_state_str);
-       os_free(old_state_str);
-}
-
-
 /**
  * wpas_dbus_signal_network_enabled_changed - Signals Enabled property changes
  * @wpa_s: %wpa_supplicant network interface data
@@ -474,17 +405,13 @@ void wpas_dbus_signal_network_enabled_changed(struct wpa_supplicant *wpa_s,
                                              struct wpa_ssid *ssid)
 {
 
-       struct network_handler_args args = { wpa_s, ssid };
        char path[WPAS_DBUS_OBJECT_PATH_MAX];
        os_snprintf(path, WPAS_DBUS_OBJECT_PATH_MAX,
                    "%s/" WPAS_DBUS_NEW_NETWORKS_PART "/%d",
                    wpa_s->dbus_new_path, ssid->id);
 
-       wpa_dbus_signal_property_changed(wpa_s->global->dbus,
-                                        (WPADBusPropertyAccessor)
-                                        wpas_dbus_getter_enabled, &args,
-                                        path, WPAS_DBUS_NEW_IFACE_NETWORK,
-                                        "Enabled");
+       wpa_dbus_mark_property_changed(wpa_s->global->dbus, path,
+                                      WPAS_DBUS_NEW_IFACE_NETWORK, "Enabled");
 }
 
 
@@ -738,6 +665,9 @@ void wpas_dbus_signal_prop_changed(struct wpa_supplicant *wpa_s,
        WPADBusPropertyAccessor getter;
        char *prop;
 
+       if (wpa_s->dbus_new_path == NULL)
+               return; /* Skip signal since D-Bus setup is not yet ready */
+
        switch (property) {
        case WPAS_DBUS_PROP_AP_SCAN:
                getter = (WPADBusPropertyAccessor) wpas_dbus_getter_ap_scan;
@@ -747,6 +677,10 @@ void wpas_dbus_signal_prop_changed(struct wpa_supplicant *wpa_s,
                getter = (WPADBusPropertyAccessor) wpas_dbus_getter_scanning;
                prop = "Scanning";
                break;
+       case WPAS_DBUS_PROP_STATE:
+               getter = (WPADBusPropertyAccessor) wpas_dbus_getter_state;
+               prop = "State";
+               break;
        case WPAS_DBUS_PROP_CURRENT_BSS:
                getter = (WPADBusPropertyAccessor)
                        wpas_dbus_getter_current_bss;
@@ -763,9 +697,65 @@ void wpas_dbus_signal_prop_changed(struct wpa_supplicant *wpa_s,
                return;
        }
 
-       wpa_dbus_signal_property_changed(wpa_s->global->dbus,
-                                        getter, wpa_s, wpa_s->dbus_new_path,
-                                        WPAS_DBUS_NEW_IFACE_INTERFACE, prop);
+       wpa_dbus_mark_property_changed(wpa_s->global->dbus,
+                                      wpa_s->dbus_new_path,
+                                      WPAS_DBUS_NEW_IFACE_INTERFACE, prop);
+}
+
+
+/**
+ * wpas_dbus_bss_signal_prop_changed - Signals change of BSS property
+ * @wpa_s: %wpa_supplicant network interface data
+ * @property: indicates which property has changed
+ * @id: unique BSS identifier
+ *
+ * Sends PropertyChanged signals with path, interface, and arguments depending
+ * on which property has changed.
+ */
+void wpas_dbus_bss_signal_prop_changed(struct wpa_supplicant *wpa_s,
+                                      enum wpas_dbus_bss_prop property,
+                                      unsigned int id)
+{
+       char path[WPAS_DBUS_OBJECT_PATH_MAX];
+       char *prop;
+
+       switch (property) {
+       case WPAS_DBUS_BSS_PROP_SIGNAL:
+               prop = "Signal";
+               break;
+       case WPAS_DBUS_BSS_PROP_FREQ:
+               prop = "Frequency";
+               break;
+       case WPAS_DBUS_BSS_PROP_MODE:
+               prop = "Mode";
+               break;
+       case WPAS_DBUS_BSS_PROP_PRIVACY:
+               prop = "Privacy";
+               break;
+       case WPAS_DBUS_BSS_PROP_RATES:
+               prop = "Rates";
+               break;
+       case WPAS_DBUS_BSS_PROP_WPA:
+               prop = "WPA";
+               break;
+       case WPAS_DBUS_BSS_PROP_RSN:
+               prop = "RSN";
+               break;
+       case WPAS_DBUS_BSS_PROP_IES:
+               prop = "IEs";
+               break;
+       default:
+               wpa_printf(MSG_ERROR, "dbus: %s: Unknown Property value %d",
+                          __func__, property);
+               return;
+       }
+
+       os_snprintf(path, WPAS_DBUS_OBJECT_PATH_MAX,
+                   "%s/" WPAS_DBUS_NEW_BSSIDS_PART "/%u",
+                   wpa_s->dbus_new_path, id);
+
+       wpa_dbus_mark_property_changed(wpa_s->global->dbus, path,
+                                      WPAS_DBUS_NEW_IFACE_BSS, prop);
 }
 
 
@@ -777,12 +767,9 @@ void wpas_dbus_signal_prop_changed(struct wpa_supplicant *wpa_s,
  */
 void wpas_dbus_signal_debug_level_changed(struct wpa_global *global)
 {
-       wpa_dbus_signal_property_changed(global->dbus,
-                                        (WPADBusPropertyAccessor)
-                                        wpas_dbus_getter_debug_level,
-                                        global, WPAS_DBUS_NEW_PATH,
-                                        WPAS_DBUS_NEW_INTERFACE,
-                                        "DebugLevel");
+       wpa_dbus_mark_property_changed(global->dbus, WPAS_DBUS_NEW_PATH,
+                                      WPAS_DBUS_NEW_INTERFACE,
+                                      "DebugLevel");
 }
 
 
@@ -794,12 +781,9 @@ void wpas_dbus_signal_debug_level_changed(struct wpa_global *global)
  */
 void wpas_dbus_signal_debug_timestamp_changed(struct wpa_global *global)
 {
-       wpa_dbus_signal_property_changed(global->dbus,
-                                        (WPADBusPropertyAccessor)
-                                        wpas_dbus_getter_debug_timestamp,
-                                        global, WPAS_DBUS_NEW_PATH,
-                                        WPAS_DBUS_NEW_INTERFACE,
-                                        "DebugTimestamp");
+       wpa_dbus_mark_property_changed(global->dbus, WPAS_DBUS_NEW_PATH,
+                                      WPAS_DBUS_NEW_INTERFACE,
+                                      "DebugTimestamp");
 }
 
 
@@ -811,73 +795,38 @@ void wpas_dbus_signal_debug_timestamp_changed(struct wpa_global *global)
  */
 void wpas_dbus_signal_debug_show_keys_changed(struct wpa_global *global)
 {
-       wpa_dbus_signal_property_changed(global->dbus,
-                                        (WPADBusPropertyAccessor)
-                                        wpas_dbus_getter_debug_show_keys,
-                                        global, WPAS_DBUS_NEW_PATH,
-                                        WPAS_DBUS_NEW_INTERFACE,
-                                        "DebugShowKeys");
+       wpa_dbus_mark_property_changed(global->dbus, WPAS_DBUS_NEW_PATH,
+                                      WPAS_DBUS_NEW_INTERFACE,
+                                      "DebugShowKeys");
 }
 
 
-struct wpas_dbus_method {
-       const char *name;
-       const char *iface;
-       WPADBusMethodHandler handler;
-       struct wpa_dbus_argument args[3];
-};
-
-struct wpas_dbus_property {
-       const char *name;
-       const char *iface;
-       const char *type;
-       WPADBusPropertyAccessor getter;
-       WPADBusPropertyAccessor setter;
-       enum dbus_prop_access _access;
-};
-
-struct wpas_dbus_signal {
-       const char *name;
-       const char *iface;
-       struct wpa_dbus_argument args[3];
-};
-
-
 static void wpas_dbus_register(struct wpa_dbus_object_desc *obj_desc,
                               void *priv,
                               WPADBusArgumentFreeFunction priv_free,
-                              const struct wpas_dbus_method *methods,
-                              const struct wpas_dbus_property *properties,
-                              const struct wpas_dbus_signal *signals)
+                              const struct wpa_dbus_method_desc *methods,
+                              const struct wpa_dbus_property_desc *properties,
+                              const struct wpa_dbus_signal_desc *signals)
 {
-       int i;
+       int n;
 
        obj_desc->user_data = priv;
        obj_desc->user_data_free_func = priv_free;
+       obj_desc->methods = methods;
+       obj_desc->properties = properties;
+       obj_desc->signals = signals;
 
-       for (i = 0; methods && methods[i].name; i++) {
-               wpa_dbus_method_register(obj_desc, methods[i].iface,
-                                        methods[i].name, methods[i].handler,
-                                        methods[i].args);
-       }
-
-       for (i = 0; properties && properties[i].name; i++) {
-               wpa_dbus_property_register(obj_desc, properties[i].iface,
-                                          properties[i].name,
-                                          properties[i].type,
-                                          properties[i].getter,
-                                          properties[i].setter,
-                                          properties[i]._access);
-       }
+       for (n = 0; properties && properties->dbus_property; properties++)
+               n++;
 
-       for (i = 0; signals && signals[i].name; i++) {
-               wpa_dbus_signal_register(obj_desc, signals[i].iface,
-                                        signals[i].name, signals[i].args);
-       }
+       obj_desc->prop_changed_flags = os_zalloc(n);
+       if (!obj_desc->prop_changed_flags)
+               wpa_printf(MSG_DEBUG, "dbus: %s: can't register handlers",
+                          __func__);
 }
 
 
-static const struct wpas_dbus_method wpas_dbus_global_methods[] = {
+static const struct wpa_dbus_method_desc wpas_dbus_global_methods[] = {
        { "CreateInterface", WPAS_DBUS_NEW_INTERFACE,
          (WPADBusMethodHandler) &wpas_dbus_handler_create_interface,
          {
@@ -904,8 +853,8 @@ static const struct wpas_dbus_method wpas_dbus_global_methods[] = {
        { NULL, NULL, NULL, { END_ARGS } }
 };
 
-static const struct wpas_dbus_property wpas_dbus_global_properties[] = {
-       { "DebugLevel", WPAS_DBUS_NEW_INTERFACE, "y",
+static const struct wpa_dbus_property_desc wpas_dbus_global_properties[] = {
+       { "DebugLevel", WPAS_DBUS_NEW_INTERFACE, "s",
          (WPADBusPropertyAccessor) wpas_dbus_getter_debug_level,
          (WPADBusPropertyAccessor) wpas_dbus_setter_debug_level,
          RW
@@ -933,7 +882,7 @@ static const struct wpas_dbus_property wpas_dbus_global_properties[] = {
        { NULL, NULL, NULL, NULL, NULL, 0 }
 };
 
-static const struct wpas_dbus_signal wpas_dbus_global_signals[] = {
+static const struct wpa_dbus_signal_desc wpas_dbus_global_signals[] = {
        { "InterfaceAdded", WPAS_DBUS_NEW_INTERFACE,
          {
                  { "path", "o", ARG_OUT },
@@ -1021,7 +970,7 @@ static void wpa_dbus_free(void *ptr)
 }
 
 
-static const struct wpas_dbus_property wpas_dbus_network_properties[] = {
+static const struct wpa_dbus_property_desc wpas_dbus_network_properties[] = {
        { "Properties", WPAS_DBUS_NEW_IFACE_NETWORK, "a{sv}",
          (WPADBusPropertyAccessor) wpas_dbus_getter_network_properties,
          (WPADBusPropertyAccessor) wpas_dbus_setter_network_properties,
@@ -1036,7 +985,7 @@ static const struct wpas_dbus_property wpas_dbus_network_properties[] = {
 };
 
 
-static const struct wpas_dbus_signal wpas_dbus_network_signals[] = {
+static const struct wpa_dbus_signal_desc wpas_dbus_network_signals[] = {
        { "PropertiesChanged", WPAS_DBUS_NEW_IFACE_NETWORK,
          {
                  { "properties", "a{sv}", ARG_OUT },
@@ -1060,10 +1009,8 @@ int wpas_dbus_register_network(struct wpa_supplicant *wpa_s,
 {
        struct wpas_dbus_priv *ctrl_iface;
        struct wpa_dbus_object_desc *obj_desc;
-
-       struct network_handler_args *arg = NULL;
-
-       char *net_obj_path;
+       struct network_handler_args *arg;
+       char net_obj_path[WPAS_DBUS_OBJECT_PATH_MAX];
 
        /* Do nothing if the control interface is not turned on */
        if (wpa_s == NULL || wpa_s->global == NULL)
@@ -1072,9 +1019,6 @@ int wpas_dbus_register_network(struct wpa_supplicant *wpa_s,
        if (ctrl_iface == NULL)
                return 0;
 
-       net_obj_path = os_zalloc(WPAS_DBUS_OBJECT_PATH_MAX);
-       if (net_obj_path == NULL)
-               return -1;
        os_snprintf(net_obj_path, WPAS_DBUS_OBJECT_PATH_MAX,
                    "%s/" WPAS_DBUS_NEW_NETWORKS_PART "/%u",
                    wpa_s->dbus_new_path, ssid->id);
@@ -1109,13 +1053,10 @@ int wpas_dbus_register_network(struct wpa_supplicant *wpa_s,
 
        wpas_dbus_signal_network_added(wpa_s, ssid->id);
 
-       os_free(net_obj_path);
        return 0;
 
 err:
-       os_free(net_obj_path);
-       os_free(obj_desc);
-       os_free(arg);
+       free_dbus_object_desc(obj_desc);
        return -1;
 }
 
@@ -1131,19 +1072,17 @@ err:
 int wpas_dbus_unregister_network(struct wpa_supplicant *wpa_s, int nid)
 {
        struct wpas_dbus_priv *ctrl_iface;
-       char *net_obj_path;
+       char net_obj_path[WPAS_DBUS_OBJECT_PATH_MAX];
        int ret;
 
        /* Do nothing if the control interface is not turned on */
-       if (wpa_s == NULL || wpa_s->global == NULL)
+       if (wpa_s == NULL || wpa_s->global == NULL ||
+           wpa_s->dbus_new_path == NULL)
                return 0;
        ctrl_iface = wpa_s->global->dbus;
        if (ctrl_iface == NULL)
                return 0;
 
-       net_obj_path = os_zalloc(WPAS_DBUS_OBJECT_PATH_MAX);
-       if (net_obj_path == NULL)
-               return -1;
        os_snprintf(net_obj_path, WPAS_DBUS_OBJECT_PATH_MAX,
                    "%s/" WPAS_DBUS_NEW_NETWORKS_PART "/%u",
                    wpa_s->dbus_new_path, nid);
@@ -1155,59 +1094,58 @@ int wpas_dbus_unregister_network(struct wpa_supplicant *wpa_s, int nid)
        if (!ret)
                wpas_dbus_signal_network_removed(wpa_s, nid);
 
-       os_free(net_obj_path);
        return ret;
 }
 
 
-static const struct wpas_dbus_property wpas_dbus_bss_properties[] = {
-       { "SSID", WPAS_DBUS_NEW_IFACE_BSSID, "ay",
+static const struct wpa_dbus_property_desc wpas_dbus_bss_properties[] = {
+       { "SSID", WPAS_DBUS_NEW_IFACE_BSS, "ay",
          (WPADBusPropertyAccessor) wpas_dbus_getter_bss_ssid,
          NULL,
          R
        },
-       { "BSSID", WPAS_DBUS_NEW_IFACE_BSSID, "ay",
+       { "BSSID", WPAS_DBUS_NEW_IFACE_BSS, "ay",
          (WPADBusPropertyAccessor) wpas_dbus_getter_bss_bssid,
          NULL,
          R
        },
-       { "Privacy", WPAS_DBUS_NEW_IFACE_BSSID, "b",
+       { "Privacy", WPAS_DBUS_NEW_IFACE_BSS, "b",
          (WPADBusPropertyAccessor) wpas_dbus_getter_bss_privacy,
          NULL,
          R
        },
-       { "Mode", WPAS_DBUS_NEW_IFACE_BSSID, "s",
+       { "Mode", WPAS_DBUS_NEW_IFACE_BSS, "s",
          (WPADBusPropertyAccessor) wpas_dbus_getter_bss_mode,
          NULL,
          R
        },
-       { "Signal", WPAS_DBUS_NEW_IFACE_BSSID, "n",
+       { "Signal", WPAS_DBUS_NEW_IFACE_BSS, "n",
          (WPADBusPropertyAccessor) wpas_dbus_getter_bss_signal,
          NULL,
          R
        },
-       { "Frequency", WPAS_DBUS_NEW_IFACE_BSSID, "q",
+       { "Frequency", WPAS_DBUS_NEW_IFACE_BSS, "q",
          (WPADBusPropertyAccessor) wpas_dbus_getter_bss_frequency,
          NULL,
          R
        },
-       { "MaxRate", WPAS_DBUS_NEW_IFACE_BSSID, "q",
-         (WPADBusPropertyAccessor) wpas_dbus_getter_bss_max_rate,
+       { "Rates", WPAS_DBUS_NEW_IFACE_BSS, "au",
+         (WPADBusPropertyAccessor) wpas_dbus_getter_bss_rates,
          NULL,
          R
        },
-       { "WPAIE", WPAS_DBUS_NEW_IFACE_BSSID, "ay",
-         (WPADBusPropertyAccessor) wpas_dbus_getter_bss_wpaie,
+       { "WPA", WPAS_DBUS_NEW_IFACE_BSS, "a{sv}",
+         (WPADBusPropertyAccessor) wpas_dbus_getter_bss_wpa,
          NULL,
          R
        },
-       { "RSNIE", WPAS_DBUS_NEW_IFACE_BSSID, "ay",
-         (WPADBusPropertyAccessor) wpas_dbus_getter_bss_rsnie,
+       { "RSN", WPAS_DBUS_NEW_IFACE_BSS, "a{sv}",
+         (WPADBusPropertyAccessor) wpas_dbus_getter_bss_rsn,
          NULL,
          R
        },
-       { "WPSIE", WPAS_DBUS_NEW_IFACE_BSSID, "ay",
-         (WPADBusPropertyAccessor) wpas_dbus_getter_bss_wpsie,
+       { "IEs", WPAS_DBUS_NEW_IFACE_BSS, "ay",
+         (WPADBusPropertyAccessor) wpas_dbus_getter_bss_ies,
          NULL,
          R
        },
@@ -1215,8 +1153,8 @@ static const struct wpas_dbus_property wpas_dbus_bss_properties[] = {
 };
 
 
-static const struct wpas_dbus_signal wpas_dbus_bss_signals[] = {
-       { "PropertiesChanged", WPAS_DBUS_NEW_IFACE_BSSID,
+static const struct wpa_dbus_signal_desc wpas_dbus_bss_signals[] = {
+       { "PropertiesChanged", WPAS_DBUS_NEW_IFACE_BSS,
          {
                  { "properties", "a{sv}", ARG_OUT },
                  END_ARGS
@@ -1239,7 +1177,7 @@ int wpas_dbus_unregister_bss(struct wpa_supplicant *wpa_s,
                             u8 bssid[ETH_ALEN], unsigned int id)
 {
        struct wpas_dbus_priv *ctrl_iface;
-       char *bss_obj_path;
+       char bss_obj_path[WPAS_DBUS_OBJECT_PATH_MAX];
 
        /* Do nothing if the control interface is not turned on */
        if (wpa_s == NULL || wpa_s->global == NULL)
@@ -1248,10 +1186,6 @@ int wpas_dbus_unregister_bss(struct wpa_supplicant *wpa_s,
        if (ctrl_iface == NULL)
                return 0;
 
-       bss_obj_path = os_zalloc(WPAS_DBUS_OBJECT_PATH_MAX);
-       if (bss_obj_path == NULL)
-               return -1;
-
        os_snprintf(bss_obj_path, WPAS_DBUS_OBJECT_PATH_MAX,
                    "%s/" WPAS_DBUS_NEW_BSSIDS_PART "/%u",
                    wpa_s->dbus_new_path, id);
@@ -1259,16 +1193,13 @@ int wpas_dbus_unregister_bss(struct wpa_supplicant *wpa_s,
        wpa_printf(MSG_DEBUG, "dbus: Unregister BSS object '%s'",
                   bss_obj_path);
        if (wpa_dbus_unregister_object_per_iface(ctrl_iface, bss_obj_path)) {
-               wpa_printf(MSG_ERROR,
-                          "Cannot unregister BSSID dbus object %s.",
+               wpa_printf(MSG_ERROR, "dbus: Cannot unregister BSS object %s",
                           bss_obj_path);
-               os_free(bss_obj_path);
                return -1;
        }
 
        wpas_dbus_signal_bss_removed(wpa_s, bss_obj_path);
 
-       os_free(bss_obj_path);
        return 0;
 }
 
@@ -1287,9 +1218,8 @@ int wpas_dbus_register_bss(struct wpa_supplicant *wpa_s,
 {
        struct wpas_dbus_priv *ctrl_iface;
        struct wpa_dbus_object_desc *obj_desc;
-       char *bss_obj_path;
-
-       struct bss_handler_args *arg = NULL;
+       char bss_obj_path[WPAS_DBUS_OBJECT_PATH_MAX];
+       struct bss_handler_args *arg;
 
        /* Do nothing if the control interface is not turned on */
        if (wpa_s == NULL || wpa_s->global == NULL)
@@ -1298,10 +1228,6 @@ int wpas_dbus_register_bss(struct wpa_supplicant *wpa_s,
        if (ctrl_iface == NULL)
                return 0;
 
-       bss_obj_path = os_zalloc(WPAS_DBUS_OBJECT_PATH_MAX);
-       if (bss_obj_path == NULL)
-               return -1;
-
        os_snprintf(bss_obj_path, WPAS_DBUS_OBJECT_PATH_MAX,
                    "%s/" WPAS_DBUS_NEW_BSSIDS_PART "/%u",
                    wpa_s->dbus_new_path, id);
@@ -1338,18 +1264,15 @@ int wpas_dbus_register_bss(struct wpa_supplicant *wpa_s,
 
        wpas_dbus_signal_bss_added(wpa_s, bss_obj_path);
 
-       os_free(bss_obj_path);
        return 0;
 
 err:
-       os_free(bss_obj_path);
-       os_free(obj_desc);
-       os_free(arg);
+       free_dbus_object_desc(obj_desc);
        return -1;
 }
 
 
-static const struct wpas_dbus_method wpas_dbus_interface_methods[] = {
+static const struct wpa_dbus_method_desc wpas_dbus_interface_methods[] = {
        { "Scan", WPAS_DBUS_NEW_IFACE_INTERFACE,
          (WPADBusMethodHandler) &wpas_dbus_handler_scan,
          {
@@ -1421,7 +1344,7 @@ static const struct wpas_dbus_method wpas_dbus_interface_methods[] = {
        { NULL, NULL, NULL, { END_ARGS } }
 };
 
-static const struct wpas_dbus_property wpas_dbus_interface_properties[] = {
+static const struct wpa_dbus_property_desc wpas_dbus_interface_properties[] = {
        { "Capabilities", WPAS_DBUS_NEW_IFACE_INTERFACE, "a{sv}",
          (WPADBusPropertyAccessor) wpas_dbus_getter_capabilities,
          NULL, R
@@ -1481,20 +1404,13 @@ static const struct wpas_dbus_property wpas_dbus_interface_properties[] = {
        { NULL, NULL, NULL, NULL, NULL, 0 }
 };
 
-static const struct wpas_dbus_signal wpas_dbus_interface_signals[] = {
+static const struct wpa_dbus_signal_desc wpas_dbus_interface_signals[] = {
        { "ScanDone", WPAS_DBUS_NEW_IFACE_INTERFACE,
          {
                  { "success", "b", ARG_OUT },
                  END_ARGS
          }
        },
-       { "StateChanged", WPAS_DBUS_NEW_IFACE_INTERFACE,
-         {
-                 { "newState", "s", ARG_OUT },
-                 { "oldState", "s", ARG_OUT },
-                 END_ARGS
-         }
-       },
        { "BSSAdded", WPAS_DBUS_NEW_IFACE_INTERFACE,
          {
                  { "path", "o", ARG_OUT },
@@ -1615,7 +1531,7 @@ int wpas_dbus_register_interface(struct wpa_supplicant *wpa_s)
 err:
        os_free(wpa_s->dbus_new_path);
        wpa_s->dbus_new_path = NULL;
-       os_free(obj_desc);
+       free_dbus_object_desc(obj_desc);
        return -1;
 }