]> git.ipfire.org Git - thirdparty/hostap.git/blobdiff - wpa_supplicant/dbus/dbus_new_handlers_p2p.c
P2P: Allow p2p_invite-persistent to specify channel for GO
[thirdparty/hostap.git] / wpa_supplicant / dbus / dbus_new_handlers_p2p.c
index 7ae1173723bae9dcd64515524e6c1849d216c39b..30e0eb3e5fb8c7f215057c59329168f1ec324aa7 100644 (file)
@@ -1,14 +1,9 @@
 /*
  * WPA Supplicant / dbus-based control interface (P2P)
+ * Copyright (c) 2011-2012, Intel Corporation
  *
- * This program is free software; you can redistribute it and/or modify
- * it under the terms of the GNU General Public License version 2 as
- * published by the Free Software Foundation.
- *
- * Alternatively, this software may be distributed under the terms of BSD
- * license.
- *
- * See README and COPYING for more details.
+ * This software may be distributed under the terms of the BSD license.
+ * See README for more details.
  */
 
 #include "includes.h"
@@ -45,7 +40,7 @@ static int parse_peer_object_path(char *peer_path, u8 addr[ETH_ALEN])
 
        if (!peer_path)
                return -1;
-       p = strrchr(peer_path, '/');
+       p = os_strrchr(peer_path, '/');
        if (!p)
                return -1;
        p++;
@@ -78,7 +73,6 @@ DBusMessage * wpas_dbus_handler_p2p_find(DBusMessage *message,
        DBusMessageIter iter;
        DBusMessageIter iter_dict;
        unsigned int timeout = 0;
-       unsigned int searchonly = 0;
        enum p2p_discovery_type type = P2P_FIND_ONLY_SOCIAL;
        int num_req_dev_types = 0;
        unsigned int i;
@@ -87,7 +81,7 @@ DBusMessage * wpas_dbus_handler_p2p_find(DBusMessage *message,
        dbus_message_iter_init(message, &iter);
        entry.key = NULL;
 
-       if (!wpa_dbus_dict_open_read(&iter, &iter_dict))
+       if (!wpa_dbus_dict_open_read(&iter, &iter_dict, NULL))
                goto error;
 
        while (wpa_dbus_dict_has_dict_entry(&iter_dict)) {
@@ -97,14 +91,12 @@ DBusMessage * wpas_dbus_handler_p2p_find(DBusMessage *message,
                if (!os_strcmp(entry.key, "Timeout") &&
                    (entry.type == DBUS_TYPE_INT32)) {
                        timeout = entry.uint32_value;
-               } else if (!os_strcmp(entry.key, "SearchOnly") &&
-                          (entry.type == DBUS_TYPE_BOOLEAN)) {
-                       searchonly = (entry.bool_value == TRUE) ? 1 : 0;
                } else if (os_strcmp(entry.key, "RequestedDeviceTypes") == 0) {
                        if ((entry.type != DBUS_TYPE_ARRAY) ||
                            (entry.array_type != WPAS_DBUS_TYPE_BINARRAY))
                                goto error_clear;
 
+                       os_free(req_dev_types);
                        req_dev_types =
                                os_malloc(WPS_DEV_TYPE_LEN * entry.array_len);
                        if (!req_dev_types)
@@ -118,20 +110,31 @@ DBusMessage * wpas_dbus_handler_p2p_find(DBusMessage *message,
                                          wpabuf_head(entry.binarray_value[i]),
                                          WPS_DEV_TYPE_LEN);
                        }
-
                        num_req_dev_types = entry.array_len;
+               } else if (!os_strcmp(entry.key, "DiscoveryType") &&
+                          (entry.type == DBUS_TYPE_STRING)) {
+                       if (!os_strcmp(entry.str_value, "start_with_full"))
+                               type = P2P_FIND_START_WITH_FULL;
+                       else if (!os_strcmp(entry.str_value, "social"))
+                               type = P2P_FIND_ONLY_SOCIAL;
+                       else if (!os_strcmp(entry.str_value, "progressive"))
+                               type = P2P_FIND_PROGRESSIVE;
+                       else
+                               goto error_clear;
                } else
                        goto error_clear;
                wpa_dbus_dict_entry_clear(&entry);
        }
 
-       wpas_p2p_find(wpa_s, timeout, type, num_req_dev_types, req_dev_types);
+       wpas_p2p_find(wpa_s, timeout, type, num_req_dev_types, req_dev_types,
+                     NULL, 0);
+       os_free(req_dev_types);
        return reply;
 
 error_clear:
-       os_free(req_dev_types);
        wpa_dbus_dict_entry_clear(&entry);
 error:
+       os_free(req_dev_types);
        reply = wpas_dbus_error_invalid_args(message, entry.key);
        return reply;
 }
@@ -195,7 +198,7 @@ DBusMessage * wpas_dbus_handler_p2p_extendedlisten(
        dbus_message_iter_init(message, &iter);
        entry.key = NULL;
 
-       if (!wpa_dbus_dict_open_read(&iter, &iter_dict))
+       if (!wpa_dbus_dict_open_read(&iter, &iter_dict, NULL))
                goto error;
 
        while (wpa_dbus_dict_has_dict_entry(&iter_dict)) {
@@ -237,7 +240,7 @@ DBusMessage * wpas_dbus_handler_p2p_presence_request(
        dbus_message_iter_init(message, &iter);
        entry.key = NULL;
 
-       if (!wpa_dbus_dict_open_read(&iter, &iter_dict))
+       if (!wpa_dbus_dict_open_read(&iter, &iter_dict, NULL))
                goto error;
 
        while (wpa_dbus_dict_has_dict_entry(&iter_dict)) {
@@ -291,7 +294,7 @@ DBusMessage * wpas_dbus_handler_p2p_group_add(DBusMessage *message,
 
        dbus_message_iter_init(message, &iter);
 
-       if (!wpa_dbus_dict_open_read(&iter, &iter_dict))
+       if (!wpa_dbus_dict_open_read(&iter, &iter_dict, NULL))
                goto inv_args;
 
        while (wpa_dbus_dict_has_dict_entry(&iter_dict)) {
@@ -338,18 +341,18 @@ DBusMessage * wpas_dbus_handler_p2p_group_add(DBusMessage *message,
                        goto out;
                }
 
-               /* Get the SSID structure form the persistant group id */
+               /* Get the SSID structure from the persistent group id */
                ssid = wpa_config_get_network(wpa_s->conf, group_id);
                if (ssid == NULL || ssid->disabled != 2)
                        goto inv_args;
 
-               if (wpas_p2p_group_add_persistent(wpa_s, ssid, 0, freq)) {
+               if (wpas_p2p_group_add_persistent(wpa_s, ssid, 0, freq, 0)) {
                        reply = wpas_dbus_error_unknown_error(
                                message,
                                "Failed to reinvoke a persistent group");
                        goto out;
                }
-       } else if (wpas_p2p_group_add(wpa_s, persistent_group, freq))
+       } else if (wpas_p2p_group_add(wpa_s, persistent_group, freq, 0))
                goto inv_args;
 
 out:
@@ -376,9 +379,35 @@ DBusMessage * wpas_dbus_handler_p2p_disconnect(DBusMessage *message,
 }
 
 
+static dbus_bool_t wpa_dbus_p2p_check_enabled(struct wpa_supplicant *wpa_s,
+                                             DBusMessage *message,
+                                             DBusMessage **out_reply,
+                                             DBusError *error)
+{
+       /* Return an error message or an error if P2P isn't available */
+       if (wpa_s->global->p2p_disabled || wpa_s->global->p2p == NULL) {
+               if (out_reply) {
+                       *out_reply = dbus_message_new_error(
+                               message, DBUS_ERROR_FAILED,
+                               "P2P is not available for this interface");
+               }
+               dbus_set_error_const(error, DBUS_ERROR_FAILED,
+                                    "P2P is not available for this "
+                                    "interface");
+               return FALSE;
+       }
+       return TRUE;
+}
+
+
 DBusMessage * wpas_dbus_handler_p2p_flush(DBusMessage *message,
                                          struct wpa_supplicant *wpa_s)
 {
+       DBusMessage *reply = NULL;
+
+       if (!wpa_dbus_p2p_check_enabled(wpa_s, message, &reply, NULL))
+               return reply;
+
        os_memset(wpa_s->p2p_auth_invite, 0, ETH_ALEN);
        wpa_s->force_long_sd = 0;
        p2p_flush(wpa_s->global->p2p);
@@ -407,9 +436,12 @@ DBusMessage * wpas_dbus_handler_p2p_connect(DBusMessage *message,
        char *err_msg = NULL;
        char *iface = NULL;
 
+       if (!wpa_dbus_p2p_check_enabled(wpa_s, message, &reply, NULL))
+               return reply;
+
        dbus_message_iter_init(message, &iter);
 
-       if (!wpa_dbus_dict_open_read(&iter, &iter_dict))
+       if (!wpa_dbus_dict_open_read(&iter, &iter_dict, NULL))
                goto inv_args;
 
        while (wpa_dbus_dict_has_dict_entry(&iter_dict)) {
@@ -444,8 +476,6 @@ DBusMessage * wpas_dbus_handler_p2p_connect(DBusMessage *message,
                                wps_method = WPS_PBC;
                        else if (!os_strcmp(entry.str_value, "pin"))
                                wps_method = WPS_PIN_DISPLAY;
-                       else if (!os_strcmp(entry.str_value, "label"))
-                               wps_method = WPS_PIN_LABEL;
                        else if (!os_strcmp(entry.str_value, "display"))
                                wps_method = WPS_PIN_DISPLAY;
                        else if (!os_strcmp(entry.str_value, "keypad"))
@@ -463,24 +493,27 @@ DBusMessage * wpas_dbus_handler_p2p_connect(DBusMessage *message,
 
        if (!peer_object_path || (wps_method == WPS_NOT_READY) ||
            (parse_peer_object_path(peer_object_path, addr) < 0) ||
-           (p2p_get_peer_info(wpa_s->global->p2p, addr, 0, NULL, 0) < 0))
+           !p2p_peer_known(wpa_s->global->p2p, addr))
                goto inv_args;
 
        /*
         * Validate the wps_method specified and the pin value.
         */
-       if ((!pin || !pin[0]) &&
-           ((wps_method == WPS_PIN_LABEL) || (wps_method == WPS_PIN_KEYPAD)))
+       if ((!pin || !pin[0]) && (wps_method == WPS_PIN_KEYPAD))
                goto inv_args;
 
        new_pin = wpas_p2p_connect(wpa_s, addr, pin, wps_method,
-                                  persistent_group, join, authorize_only,
-                                  go_intent, freq);
+                                  persistent_group, 0, join, authorize_only,
+                                  go_intent, freq, -1, 0, 0);
 
        if (new_pin >= 0) {
+               char npin[9];
+               char *generated_pin;
+               os_snprintf(npin, sizeof(npin), "%08d", new_pin);
+               generated_pin = npin;
                reply = dbus_message_new_method_return(message);
-               dbus_message_append_args(reply, DBUS_TYPE_INT32,
-                                        &new_pin, DBUS_TYPE_INVALID);
+               dbus_message_append_args(reply, DBUS_TYPE_STRING,
+                                        &generated_pin, DBUS_TYPE_INVALID);
        } else {
                switch (new_pin) {
                case -2:
@@ -537,9 +570,12 @@ DBusMessage * wpas_dbus_handler_p2p_invite(DBusMessage *message,
        int persistent = 0;
        struct wpa_ssid *ssid;
 
+       if (!wpa_dbus_p2p_check_enabled(wpa_s, message, &reply, NULL))
+               return reply;
+
        dbus_message_iter_init(message, &iter);
 
-       if (!wpa_dbus_dict_open_read(&iter, &iter_dict))
+       if (!wpa_dbus_dict_open_read(&iter, &iter_dict, NULL))
                goto err;
 
        while (wpa_dbus_dict_has_dict_entry(&iter_dict)) {
@@ -563,15 +599,14 @@ DBusMessage * wpas_dbus_handler_p2p_invite(DBusMessage *message,
 
        if (!peer_object_path ||
            (parse_peer_object_path(peer_object_path, peer_addr) < 0) ||
-           (p2p_get_peer_info(wpa_s->global->p2p,
-                              peer_addr, 0, NULL, 0) < 0)) {
+           !p2p_peer_known(wpa_s->global->p2p, peer_addr)) {
                goto err;
        }
 
        if (persistent) {
                /*
                 * A group ID is defined meaning we want to re-invoke a
-                * persistant group
+                * persistent group
                 */
 
                iface = wpas_dbus_new_decompose_object_path(pg_object_path, 1,
@@ -590,12 +625,12 @@ DBusMessage * wpas_dbus_handler_p2p_invite(DBusMessage *message,
                        goto out;
                }
 
-               /* Get the SSID structure form the persistant group id */
+               /* Get the SSID structure from the persistent group id */
                ssid = wpa_config_get_network(wpa_s->conf, group_id);
                if (ssid == NULL || ssid->disabled != 2)
                        goto err;
 
-               if (wpas_p2p_invite(wpa_s, peer_addr, ssid, NULL) < 0) {
+               if (wpas_p2p_invite(wpa_s, peer_addr, ssid, NULL, 0, 0) < 0) {
                        reply = wpas_dbus_error_unknown_error(
                                message,
                                "Failed to reinvoke a persistent group");
@@ -652,7 +687,8 @@ DBusMessage * wpas_dbus_handler_p2p_prov_disc_req(DBusMessage *message,
            os_strcmp(config_method, "pushbutton"))
                return wpas_dbus_error_invalid_args(message, NULL);
 
-       if (wpas_p2p_prov_disc(wpa_s, peer_addr, config_method) < 0)
+       if (wpas_p2p_prov_disc(wpa_s, peer_addr, config_method,
+                              WPAS_P2P_PD_FOR_GO_NEG) < 0)
                return wpas_dbus_error_unknown_error(message,
                                "Failed to send provision discovery request");
 
@@ -664,11 +700,12 @@ DBusMessage * wpas_dbus_handler_p2p_prov_disc_req(DBusMessage *message,
  * P2P Device property accessor methods.
  */
 
-DBusMessage * wpas_dbus_getter_p2p_device_properties(
-       DBusMessage *message, struct wpa_supplicant *wpa_s)
+dbus_bool_t wpas_dbus_getter_p2p_device_config(DBusMessageIter *iter,
+                                              DBusError *error,
+                                              void *user_data)
 {
-       DBusMessage *reply = NULL;
-       DBusMessageIter iter, variant_iter, dict_iter;
+       struct wpa_supplicant *wpa_s = user_data;
+       DBusMessageIter variant_iter, dict_iter;
        DBusMessageIter iter_secdev_dict_entry, iter_secdev_dict_val,
                iter_secdev_dict_array;
        const char *dev_name;
@@ -676,17 +713,10 @@ DBusMessage * wpas_dbus_getter_p2p_device_properties(
        int i;
        const struct wpabuf *vendor_ext[P2P_MAX_WPS_VENDOR_EXT];
 
-       if (message == NULL)
-               reply = dbus_message_new(DBUS_MESSAGE_TYPE_SIGNAL);
-       else
-               reply = dbus_message_new_method_return(message);
-
-       if (!reply)
-               goto err_no_mem;
-
-       dbus_message_iter_init_append(reply, &iter);
+       if (!wpa_dbus_p2p_check_enabled(wpa_s, NULL, NULL, error))
+               return FALSE;
 
-       if (!dbus_message_iter_open_container(&iter, DBUS_TYPE_VARIANT,
+       if (!dbus_message_iter_open_container(iter, DBUS_TYPE_VARIANT,
                                              "a{sv}", &variant_iter) ||
            !wpa_dbus_dict_open_write(&variant_iter, &dict_iter))
                goto err_no_mem;
@@ -747,8 +777,8 @@ DBusMessage * wpas_dbus_getter_p2p_device_properties(
                                         wpa_s->conf->p2p_go_intent))
                goto err_no_mem;
 
-       /* Persistant Reconnect */
-       if (!wpa_dbus_dict_append_bool(&dict_iter, "PersistantReconnect",
+       /* Persistent Reconnect */
+       if (!wpa_dbus_dict_append_bool(&dict_iter, "PersistentReconnect",
                                       wpa_s->conf->persistent_reconnect))
                goto err_no_mem;
 
@@ -794,44 +824,45 @@ DBusMessage * wpas_dbus_getter_p2p_device_properties(
                goto err_no_mem;
 
        if (!wpa_dbus_dict_close_write(&variant_iter, &dict_iter) ||
-           !dbus_message_iter_close_container(&iter, &variant_iter))
+           !dbus_message_iter_close_container(iter, &variant_iter))
                goto err_no_mem;
 
-       return reply;
+       return TRUE;
+
 err_no_mem:
-       dbus_message_unref(reply);
-       return dbus_message_new_error(message, DBUS_ERROR_NO_MEMORY, NULL);
+       dbus_set_error_const(error, DBUS_ERROR_NO_MEMORY, "no memory");
+       return FALSE;
 }
 
 
-DBusMessage * wpas_dbus_setter_p2p_device_properties(
-       DBusMessage *message, struct wpa_supplicant *wpa_s)
+dbus_bool_t wpas_dbus_setter_p2p_device_config(DBusMessageIter *iter,
+                                              DBusError *error,
+                                              void *user_data)
 {
-       DBusMessage *reply = NULL;
-       DBusMessageIter iter, variant_iter;
+       struct wpa_supplicant *wpa_s = user_data;
+       DBusMessageIter variant_iter, iter_dict;
        struct wpa_dbus_dict_entry entry = {.type = DBUS_TYPE_STRING };
-       DBusMessageIter iter_dict;
        unsigned int i;
 
-       dbus_message_iter_init(message, &iter);
-
-       dbus_message_iter_next(&iter);
-       dbus_message_iter_next(&iter);
-
-       dbus_message_iter_recurse(&iter, &variant_iter);
+       if (!wpa_dbus_p2p_check_enabled(wpa_s, NULL, NULL, error))
+               return FALSE;
 
-       if (!wpa_dbus_dict_open_read(&variant_iter, &iter_dict))
-               return wpas_dbus_error_invalid_args(message, NULL);
+       dbus_message_iter_recurse(iter, &variant_iter);
+       if (!wpa_dbus_dict_open_read(&variant_iter, &iter_dict, error))
+               return FALSE;
 
        while (wpa_dbus_dict_has_dict_entry(&iter_dict)) {
-               if (!wpa_dbus_dict_get_entry(&iter_dict, &entry))
-                       return wpas_dbus_error_invalid_args(message, NULL);
+               if (!wpa_dbus_dict_get_entry(&iter_dict, &entry)) {
+                       dbus_set_error_const(error, DBUS_ERROR_INVALID_ARGS,
+                                            "invalid message format");
+                       return FALSE;
+               }
 
                if (os_strcmp(entry.key, "DeviceName") == 0) {
                        char *devname;
 
                        if (entry.type != DBUS_TYPE_STRING)
-                               goto error_clear;
+                               goto error;
 
                        devname = os_strdup(entry.str_value);
                        if (devname == NULL)
@@ -846,7 +877,7 @@ DBusMessage * wpas_dbus_setter_p2p_device_properties(
                        if (entry.type != DBUS_TYPE_ARRAY ||
                            entry.array_type != DBUS_TYPE_BYTE ||
                            entry.array_len != WPS_DEV_TYPE_LEN)
-                               goto error_clear;
+                               goto error;
 
                        os_memcpy(wpa_s->conf->device_type,
                                  entry.bytearray_value,
@@ -874,7 +905,7 @@ DBusMessage * wpas_dbus_setter_p2p_device_properties(
                        if ((entry.type != DBUS_TYPE_ARRAY) ||
                            (entry.array_type != WPAS_DBUS_TYPE_BINARRAY) ||
                            (entry.array_len > P2P_MAX_WPS_VENDOR_EXT))
-                               goto error_clear;
+                               goto error;
 
                        wpa_s->conf->changed_parameters |=
                                CFG_CHANGED_VENDOR_EXTENSION;
@@ -892,7 +923,7 @@ DBusMessage * wpas_dbus_setter_p2p_device_properties(
                           (entry.type == DBUS_TYPE_UINT32) &&
                           (entry.uint32_value <= 15))
                        wpa_s->conf->p2p_go_intent = entry.uint32_value;
-               else if ((os_strcmp(entry.key, "PersistantReconnect") == 0) &&
+               else if ((os_strcmp(entry.key, "PersistentReconnect") == 0) &&
                         (entry.type == DBUS_TYPE_BOOLEAN))
                        wpa_s->conf->persistent_reconnect = entry.bool_value;
                else if ((os_strcmp(entry.key, "ListenRegClass") == 0) &&
@@ -919,7 +950,7 @@ DBusMessage * wpas_dbus_setter_p2p_device_properties(
                        char *postfix;
 
                        if (entry.type != DBUS_TYPE_STRING)
-                               goto error_clear;
+                               goto error;
 
                        postfix = os_strdup(entry.str_value);
                        if (!postfix)
@@ -942,7 +973,7 @@ DBusMessage * wpas_dbus_setter_p2p_device_properties(
                         entry.type == DBUS_TYPE_UINT32)
                        wpa_s->conf->disassoc_low_ack = entry.uint32_value;
                else
-                       goto error_clear;
+                       goto error;
 
                wpa_dbus_dict_entry_clear(&entry);
        }
@@ -952,30 +983,31 @@ DBusMessage * wpas_dbus_setter_p2p_device_properties(
                wpa_supplicant_update_config(wpa_s);
        }
 
-       return reply;
+       return TRUE;
 
- error_clear:
-       wpa_dbus_dict_entry_clear(&entry);
  error:
-       reply = wpas_dbus_error_invalid_args(message, entry.key);
+       dbus_set_error_const(error, DBUS_ERROR_INVALID_ARGS,
+                            "invalid message format");
        wpa_dbus_dict_entry_clear(&entry);
+       return FALSE;
 
-       return reply;
  err_no_mem_clear:
+       dbus_set_error_const(error, DBUS_ERROR_NO_MEMORY, "no memory");
        wpa_dbus_dict_entry_clear(&entry);
-       return dbus_message_new_error(message, DBUS_ERROR_NO_MEMORY, NULL);
+       return FALSE;
 }
 
 
-DBusMessage * wpas_dbus_getter_p2p_peers(DBusMessage *message,
-                                        struct wpa_supplicant *wpa_s)
+dbus_bool_t wpas_dbus_getter_p2p_peers(DBusMessageIter *iter, DBusError *error,
+                                      void *user_data)
 {
-       DBusMessage *reply = NULL;
+       struct wpa_supplicant *wpa_s = user_data;
        struct p2p_data *p2p = wpa_s->global->p2p;
        int next = 0, i = 0;
        int num = 0, out_of_mem = 0;
        const u8 *addr;
        const struct p2p_peer_info *peer_info = NULL;
+       dbus_bool_t success = FALSE;
 
        struct dl_list peer_objpath_list;
        struct peer_objpath_node {
@@ -985,6 +1017,9 @@ DBusMessage * wpas_dbus_getter_p2p_peers(DBusMessage *message,
 
        char **peer_obj_paths = NULL;
 
+       if (!wpa_dbus_p2p_check_enabled(wpa_s, NULL, NULL, error))
+               return FALSE;
+
        dl_list_init(&peer_objpath_list);
 
        /* Get the first peer info */
@@ -1014,7 +1049,7 @@ DBusMessage * wpas_dbus_getter_p2p_peers(DBusMessage *message,
         * Now construct the peer object paths in a form suitable for
         * array_property_getter helper below.
         */
-       peer_obj_paths = os_zalloc(num * sizeof(char *));
+       peer_obj_paths = os_calloc(num, sizeof(char *));
 
        if (!peer_obj_paths) {
                out_of_mem = 1;
@@ -1025,9 +1060,10 @@ DBusMessage * wpas_dbus_getter_p2p_peers(DBusMessage *message,
                              struct peer_objpath_node, list)
                peer_obj_paths[i++] = node->path;
 
-       reply = wpas_dbus_simple_array_property_getter(message,
-                                                      DBUS_TYPE_OBJECT_PATH,
-                                                      peer_obj_paths, num);
+       success = wpas_dbus_simple_array_property_getter(iter,
+                                                        DBUS_TYPE_OBJECT_PATH,
+                                                        peer_obj_paths, num,
+                                                        error);
 
 error:
        if (peer_obj_paths)
@@ -1039,10 +1075,9 @@ error:
                os_free(node);
        }
        if (out_of_mem)
-               reply = dbus_message_new_error(message, DBUS_ERROR_NO_MEMORY,
-                                              NULL);
+               dbus_set_error_const(error, DBUS_ERROR_NO_MEMORY, "no memory");
 
-       return reply;
+       return success;
 }
 
 
@@ -1075,9 +1110,10 @@ static enum wpas_p2p_role wpas_get_p2p_role(struct wpa_supplicant *wpa_s)
 }
 
 
-DBusMessage * wpas_dbus_getter_p2p_role(DBusMessage *message,
-                                       struct wpa_supplicant *wpa_s)
+dbus_bool_t wpas_dbus_getter_p2p_role(DBusMessageIter *iter, DBusError *error,
+                                     void *user_data)
 {
+       struct wpa_supplicant *wpa_s = user_data;
        char *str;
 
        switch (wpas_get_p2p_role(wpa_s)) {
@@ -1091,37 +1127,47 @@ DBusMessage * wpas_dbus_getter_p2p_role(DBusMessage *message,
                str = "device";
        }
 
-       return wpas_dbus_simple_property_getter(message, DBUS_TYPE_STRING,
-                                               &str);
+       return wpas_dbus_simple_property_getter(iter, DBUS_TYPE_STRING, &str,
+                                               error);
 }
 
 
-DBusMessage * wpas_dbus_getter_p2p_group(DBusMessage *message,
-                                       struct wpa_supplicant *wpa_s)
+dbus_bool_t wpas_dbus_getter_p2p_group(DBusMessageIter *iter, DBusError *error,
+                                      void *user_data)
 {
+       struct wpa_supplicant *wpa_s = user_data;
+       char path_buf[WPAS_DBUS_OBJECT_PATH_MAX];
+       char *dbus_groupobj_path = path_buf;
+
        if (wpa_s->dbus_groupobj_path == NULL)
-               return NULL;
+               os_snprintf(dbus_groupobj_path, WPAS_DBUS_OBJECT_PATH_MAX,
+                           "/");
+       else
+               os_snprintf(dbus_groupobj_path, WPAS_DBUS_OBJECT_PATH_MAX,
+                           "%s", wpa_s->dbus_groupobj_path);
 
-       return wpas_dbus_simple_property_getter(message,
-                                               DBUS_TYPE_OBJECT_PATH,
-                                               &wpa_s->dbus_groupobj_path);
+       return wpas_dbus_simple_property_getter(iter, DBUS_TYPE_OBJECT_PATH,
+                                               &dbus_groupobj_path, error);
 }
 
 
-DBusMessage * wpas_dbus_getter_p2p_peergo(DBusMessage *message,
-                                         struct wpa_supplicant *wpa_s)
+dbus_bool_t wpas_dbus_getter_p2p_peergo(DBusMessageIter *iter,
+                                       DBusError *error, void *user_data)
 {
+       struct wpa_supplicant *wpa_s = user_data;
        char go_peer_obj_path[WPAS_DBUS_OBJECT_PATH_MAX], *path;
 
        if (wpas_get_p2p_role(wpa_s) != WPAS_P2P_ROLE_CLIENT)
-               return NULL;
+               os_snprintf(go_peer_obj_path, WPAS_DBUS_OBJECT_PATH_MAX, "/");
+       else
+               os_snprintf(go_peer_obj_path, WPAS_DBUS_OBJECT_PATH_MAX,
+                           "%s/" WPAS_DBUS_NEW_P2P_PEERS_PART "/"
+                           COMPACT_MACSTR,
+                           wpa_s->dbus_new_path, MAC2STR(wpa_s->go_dev_addr));
 
-       os_snprintf(go_peer_obj_path, WPAS_DBUS_OBJECT_PATH_MAX,
-                   "%s/" WPAS_DBUS_NEW_P2P_PEERS_PART "/" COMPACT_MACSTR,
-                   wpa_s->dbus_new_path, MAC2STR(wpa_s->go_dev_addr));
        path = go_peer_obj_path;
-       return wpas_dbus_simple_property_getter(message,
-                                               DBUS_TYPE_OBJECT_PATH, &path);
+       return wpas_dbus_simple_property_getter(iter, DBUS_TYPE_OBJECT_PATH,
+                                               &path, error);
 }
 
 
@@ -1129,108 +1175,279 @@ DBusMessage * wpas_dbus_getter_p2p_peergo(DBusMessage *message,
  * Peer object properties accessor methods
  */
 
-DBusMessage * wpas_dbus_getter_p2p_peer_properties(
-       DBusMessage *message, struct peer_handler_args *peer_args)
+dbus_bool_t wpas_dbus_getter_p2p_peer_device_name(DBusMessageIter *iter,
+                                                 DBusError *error,
+                                                 void *user_data)
 {
-       DBusMessage *reply = NULL;
-       DBusMessageIter iter, variant_iter, dict_iter;
-       const struct p2p_peer_info *info = NULL;
-       char devtype[WPS_DEV_TYPE_BUFSIZE];
-       const struct wpabuf *vendor_extension[P2P_MAX_WPS_VENDOR_EXT];
-       int i, num;
+       struct peer_handler_args *peer_args = user_data;
+       const struct p2p_peer_info *info;
+       char *tmp;
+
+       if (!wpa_dbus_p2p_check_enabled(peer_args->wpa_s, NULL, NULL, error))
+               return FALSE;
 
        /* get the peer info */
        info = p2p_get_peer_found(peer_args->wpa_s->global->p2p,
                                  peer_args->p2p_device_addr, 0);
-       if (info == NULL)
-               return NULL;
+       if (info == NULL) {
+               dbus_set_error(error, DBUS_ERROR_FAILED,
+                              "failed to find peer");
+               return FALSE;
+       }
 
-       if (message == NULL)
-               reply = dbus_message_new(DBUS_MESSAGE_TYPE_SIGNAL);
-       else
-               reply = dbus_message_new_method_return(message);
+       tmp = os_strdup(info->device_name);
+       if (!tmp) {
+               dbus_set_error_const(error, DBUS_ERROR_NO_MEMORY, "no memory");
+               return FALSE;
+       }
 
-       if (!reply)
-               goto err_no_mem;
+       if (!wpas_dbus_simple_property_getter(iter, DBUS_TYPE_STRING, &tmp,
+                                             error)) {
+               dbus_set_error_const(error, DBUS_ERROR_NO_MEMORY, "no memory");
+               os_free(tmp);
+               return FALSE;
+       }
 
-       dbus_message_iter_init_append(reply, &iter);
-       if (!dbus_message_iter_open_container(&iter, DBUS_TYPE_VARIANT,
-                                             "a{sv}", &variant_iter) ||
-           !wpa_dbus_dict_open_write(&variant_iter, &dict_iter))
-               goto err_no_mem;
+       os_free(tmp);
+       return TRUE;
+}
 
-       /* Fill out the dictionary */
-       wps_dev_type_bin2str(info->pri_dev_type, devtype, sizeof(devtype));
-       if (!wpa_dbus_dict_append_string(&dict_iter, "DeviceName",
-                                        info->device_name))
-               goto err_no_mem;
-       if (!wpa_dbus_dict_append_string(&dict_iter, "PrimaryDeviceType",
-                                        devtype))
-               goto err_no_mem;
-       if (!wpa_dbus_dict_append_uint16(&dict_iter, "config_method",
-                                        info->config_methods))
-               goto err_no_mem;
-       if (!wpa_dbus_dict_append_int32(&dict_iter, "level",
-                                       info->level))
-               goto err_no_mem;
-       if (!wpa_dbus_dict_append_byte(&dict_iter, "devicecapability",
-                                      info->dev_capab))
-               goto err_no_mem;
-       if (!wpa_dbus_dict_append_byte(&dict_iter, "groupcapability",
-                                      info->group_capab))
-               goto err_no_mem;
 
-       if (info->wps_sec_dev_type_list_len) {
-               char *sec_dev_types[MAX_SEC_DEVICE_TYPES];
-               u8 *sec_dev_type_list = NULL;
-               char secdevtype[WPS_DEV_TYPE_BUFSIZE];
-               int num_sec_dev_types = 0;
+dbus_bool_t wpas_dbus_getter_p2p_peer_primary_device_type(
+       DBusMessageIter *iter, DBusError *error, void *user_data)
+{
+       struct peer_handler_args *peer_args = user_data;
+       const struct p2p_peer_info *info;
 
-               sec_dev_type_list = os_zalloc(info->wps_sec_dev_type_list_len);
+       info = p2p_get_peer_found(peer_args->wpa_s->global->p2p,
+                                 peer_args->p2p_device_addr, 0);
+       if (info == NULL) {
+               dbus_set_error(error, DBUS_ERROR_FAILED,
+                              "failed to find peer");
+               return FALSE;
+       }
 
-               if (sec_dev_type_list == NULL)
-                       goto err_no_mem;
+       if (!wpas_dbus_simple_array_property_getter(iter, DBUS_TYPE_BYTE,
+                                                   (char *)
+                                                   info->pri_dev_type,
+                                                   WPS_DEV_TYPE_LEN, error)) {
+               dbus_set_error_const(error, DBUS_ERROR_NO_MEMORY, "no memory");
+               return FALSE;
+       }
 
-               os_memcpy(sec_dev_type_list, info->wps_sec_dev_type_list,
-                         info->wps_sec_dev_type_list_len);
+       return TRUE;
+}
 
-               for (i = 0; i < MAX_SEC_DEVICE_TYPES &&
-                      i < (int) (info->wps_sec_dev_type_list_len /
-                                 WPS_DEV_TYPE_LEN);
-                    i++) {
-                       sec_dev_types[i] = os_zalloc(sizeof(secdevtype));
-
-                       if (!sec_dev_types[i] ||
-                           wps_dev_type_bin2str(
-                                   &sec_dev_type_list[i * WPS_DEV_TYPE_LEN],
-                                   sec_dev_types[i],
-                                   sizeof(secdevtype)) == NULL) {
-                               while (--i >= 0)
-                                       os_free(sec_dev_types[i]);
-                               os_free(sec_dev_type_list);
-                               goto err_no_mem;
-                       }
 
-                       num_sec_dev_types++;
-               }
+dbus_bool_t wpas_dbus_getter_p2p_peer_config_method(DBusMessageIter *iter,
+                                                    DBusError *error,
+                                                    void *user_data)
+{
+       struct peer_handler_args *peer_args = user_data;
+       const struct p2p_peer_info *info;
 
-               os_free(sec_dev_type_list);
+       info = p2p_get_peer_found(peer_args->wpa_s->global->p2p,
+                                 peer_args->p2p_device_addr, 0);
+       if (info == NULL) {
+               dbus_set_error(error, DBUS_ERROR_FAILED,
+                              "failed to find peer");
+               return FALSE;
+       }
+
+       if (!wpas_dbus_simple_property_getter(iter, DBUS_TYPE_UINT16,
+                                             &info->config_methods, error)) {
+               dbus_set_error_const(error, DBUS_ERROR_NO_MEMORY, "no memory");
+               return FALSE;
+       }
+
+       return TRUE;
+}
+
+
+dbus_bool_t wpas_dbus_getter_p2p_peer_level(DBusMessageIter *iter,
+                                            DBusError *error,
+                                            void *user_data)
+{
+       struct peer_handler_args *peer_args = user_data;
+       const struct p2p_peer_info *info;
 
-               if (num_sec_dev_types) {
-                       if (!wpa_dbus_dict_append_string_array(&dict_iter,
-                                               "SecondaryDeviceTypes",
-                                               (const char **)sec_dev_types,
-                                               num_sec_dev_types)) {
-                               for (i = 0; i < num_sec_dev_types; i++)
-                                       os_free(sec_dev_types[i]);
-                               goto err_no_mem;
+       info = p2p_get_peer_found(peer_args->wpa_s->global->p2p,
+                                 peer_args->p2p_device_addr, 0);
+       if (info == NULL) {
+               dbus_set_error(error, DBUS_ERROR_FAILED,
+                              "failed to find peer");
+               return FALSE;
+       }
+
+       if (!wpas_dbus_simple_property_getter(iter, DBUS_TYPE_INT32,
+                                             &info->level, error)) {
+               dbus_set_error_const(error, DBUS_ERROR_NO_MEMORY, "no memory");
+               return FALSE;
+       }
+
+       return TRUE;
+}
+
+
+dbus_bool_t wpas_dbus_getter_p2p_peer_device_capability(DBusMessageIter *iter,
+                                                        DBusError *error,
+                                                        void *user_data)
+{
+       struct peer_handler_args *peer_args = user_data;
+       const struct p2p_peer_info *info;
+
+       info = p2p_get_peer_found(peer_args->wpa_s->global->p2p,
+                                 peer_args->p2p_device_addr, 0);
+       if (info == NULL) {
+               dbus_set_error(error, DBUS_ERROR_FAILED,
+                              "failed to find peer");
+               return FALSE;
+       }
+
+       if (!wpas_dbus_simple_property_getter(iter, DBUS_TYPE_BYTE,
+                                             &info->dev_capab, error)) {
+               dbus_set_error_const(error, DBUS_ERROR_NO_MEMORY, "no memory");
+               return FALSE;
+       }
+
+       return TRUE;
+}
+
+
+dbus_bool_t wpas_dbus_getter_p2p_peer_group_capability(DBusMessageIter *iter,
+                                                      DBusError *error,
+                                                      void *user_data)
+{
+       struct peer_handler_args *peer_args = user_data;
+       const struct p2p_peer_info *info;
+
+       info = p2p_get_peer_found(peer_args->wpa_s->global->p2p,
+                                 peer_args->p2p_device_addr, 0);
+       if (info == NULL) {
+               dbus_set_error(error, DBUS_ERROR_FAILED,
+                              "failed to find peer");
+               return FALSE;
+       }
+
+       if (!wpas_dbus_simple_property_getter(iter, DBUS_TYPE_BYTE,
+                                             &info->group_capab, error)) {
+               dbus_set_error_const(error, DBUS_ERROR_NO_MEMORY, "no memory");
+               return FALSE;
+       }
+
+       return TRUE;
+}
+
+
+dbus_bool_t wpas_dbus_getter_p2p_peer_secondary_device_types(
+       DBusMessageIter *iter, DBusError *error, void *user_data)
+{
+       struct peer_handler_args *peer_args = user_data;
+       const struct p2p_peer_info *info;
+       DBusMessageIter variant_iter, array_iter;
+
+       info = p2p_get_peer_found(peer_args->wpa_s->global->p2p,
+                                 peer_args->p2p_device_addr, 0);
+       if (info == NULL) {
+               dbus_set_error(error, DBUS_ERROR_FAILED,
+                              "failed to find peer");
+               return FALSE;
+       }
+
+       if (!dbus_message_iter_open_container(iter, DBUS_TYPE_VARIANT,
+                                             DBUS_TYPE_ARRAY_AS_STRING
+                                             DBUS_TYPE_ARRAY_AS_STRING
+                                             DBUS_TYPE_BYTE_AS_STRING,
+                                             &variant_iter)) {
+               dbus_set_error(error, DBUS_ERROR_FAILED,
+                              "%s: failed to construct message 1", __func__);
+               return FALSE;
+       }
+
+       if (!dbus_message_iter_open_container(&variant_iter, DBUS_TYPE_ARRAY,
+                                             DBUS_TYPE_ARRAY_AS_STRING
+                                             DBUS_TYPE_BYTE_AS_STRING,
+                                             &array_iter)) {
+               dbus_set_error(error, DBUS_ERROR_FAILED,
+                              "%s: failed to construct message 2", __func__);
+               return FALSE;
+       }
+
+       if (info->wps_sec_dev_type_list_len) {
+               const u8 *sec_dev_type_list = info->wps_sec_dev_type_list;
+               int num_sec_device_types =
+                       info->wps_sec_dev_type_list_len / WPS_DEV_TYPE_LEN;
+               int i;
+               DBusMessageIter inner_array_iter;
+
+               for (i = 0; i < num_sec_device_types; i++) {
+                       if (!dbus_message_iter_open_container(
+                                   &array_iter, DBUS_TYPE_ARRAY,
+                                   DBUS_TYPE_BYTE_AS_STRING,
+                                   &inner_array_iter)) {
+                               dbus_set_error(error, DBUS_ERROR_FAILED,
+                                              "%s: failed to construct "
+                                              "message 3 (%d)",
+                                              __func__, i);
+                               return FALSE;
                        }
 
-                       for (i = 0; i < num_sec_dev_types; i++)
-                               os_free(sec_dev_types[i]);
+                       if (!dbus_message_iter_append_fixed_array(
+                                   &inner_array_iter, DBUS_TYPE_BYTE,
+                                   &sec_dev_type_list, WPS_DEV_TYPE_LEN)) {
+                               dbus_set_error(error, DBUS_ERROR_FAILED,
+                                              "%s: failed to construct "
+                                              "message 4 (%d)",
+                                              __func__, i);
+                               return FALSE;
+                       }
+
+                       if (!dbus_message_iter_close_container(
+                                   &array_iter, &inner_array_iter)) {
+                               dbus_set_error(error, DBUS_ERROR_FAILED,
+                                              "%s: failed to construct "
+                                              "message 5 (%d)",
+                                              __func__, i);
+                               return FALSE;
+                       }
+
+                       sec_dev_type_list += WPS_DEV_TYPE_LEN;
                }
        }
 
+       if (!dbus_message_iter_close_container(&variant_iter, &array_iter)) {
+               dbus_set_error(error, DBUS_ERROR_FAILED,
+                              "%s: failed to construct message 6", __func__);
+               return FALSE;
+       }
+
+       if (!dbus_message_iter_close_container(iter, &variant_iter)) {
+               dbus_set_error(error, DBUS_ERROR_FAILED,
+                              "%s: failed to construct message 7", __func__);
+               return FALSE;
+       }
+
+       return TRUE;
+}
+
+
+dbus_bool_t wpas_dbus_getter_p2p_peer_vendor_extension(DBusMessageIter *iter,
+                                                      DBusError *error,
+                                                      void *user_data)
+{
+       struct wpabuf *vendor_extension[P2P_MAX_WPS_VENDOR_EXT];
+       int i, num;
+       struct peer_handler_args *peer_args = user_data;
+       const struct p2p_peer_info *info;
+
+       info = p2p_get_peer_found(peer_args->wpa_s->global->p2p,
+                                 peer_args->p2p_device_addr, 0);
+       if (info == NULL) {
+               dbus_set_error(error, DBUS_ERROR_FAILED,
+                              "failed to find peer");
+               return FALSE;
+       }
+
        /* Add WPS vendor extensions attribute */
        for (i = 0, num = 0; i < P2P_MAX_WPS_VENDOR_EXT; i++) {
                if (info->wps_vendor_ext[i] == NULL)
@@ -1239,60 +1456,63 @@ DBusMessage * wpas_dbus_getter_p2p_peer_properties(
                num++;
        }
 
-       if (!wpa_dbus_dict_append_wpabuf_array(&dict_iter, "VendorExtension",
-                                              vendor_extension, num))
-               goto err_no_mem;
-
-       if (!wpa_dbus_dict_close_write(&variant_iter, &dict_iter) ||
-           !dbus_message_iter_close_container(&iter, &variant_iter))
-               goto err_no_mem;
+       if (!wpas_dbus_simple_array_array_property_getter(iter, DBUS_TYPE_BYTE,
+                                                         vendor_extension,
+                                                         num, error))
+               return FALSE;
 
-       return reply;
-err_no_mem:
-       dbus_message_unref(reply);
-       return dbus_message_new_error(message, DBUS_ERROR_NO_MEMORY, NULL);
+       return TRUE;
 }
 
 
-DBusMessage * wpas_dbus_getter_p2p_peer_ies(
-       DBusMessage *message, struct peer_handler_args *peer_args)
+dbus_bool_t wpas_dbus_getter_p2p_peer_ies(DBusMessageIter *iter,
+                                         DBusError *error, void *user_data)
 {
-       return NULL;
+       dbus_bool_t success;
+       /* struct peer_handler_args *peer_args = user_data; */
+
+       success = wpas_dbus_simple_array_property_getter(iter, DBUS_TYPE_BYTE,
+                                                        NULL, 0, error);
+       return success;
 }
 
 
 /**
- * wpas_dbus_getter_persistent_groups - Get array of peristent group objects
- * @message: Pointer to incoming dbus message
- * @wpa_s: wpa_supplicant structure for a network interface
- * Returns: a dbus message containing an array of all persistent group
- * dbus object paths.
+ * wpas_dbus_getter_persistent_groups - Get array of persistent group objects
+ * @iter: Pointer to incoming dbus message iter
+ * @error: Location to store error on failure
+ * @user_data: Function specific data
+ * Returns: TRUE on success, FALSE on failure
  *
- * Getter for "Networks" property.
+ * Getter for "PersistentGroups" property.
  */
-DBusMessage * wpas_dbus_getter_persistent_groups(DBusMessage *message,
-                                                struct wpa_supplicant *wpa_s)
+dbus_bool_t wpas_dbus_getter_persistent_groups(DBusMessageIter *iter,
+                                              DBusError *error,
+                                              void *user_data)
 {
-       DBusMessage *reply = NULL;
+       struct wpa_supplicant *wpa_s = user_data;
        struct wpa_ssid *ssid;
        char **paths;
        unsigned int i = 0, num = 0;
+       dbus_bool_t success = FALSE;
 
        if (wpa_s->conf == NULL) {
                wpa_printf(MSG_ERROR, "dbus: %s: "
                           "An error occurred getting persistent groups list",
                           __func__);
-               return wpas_dbus_error_unknown_error(message, NULL);
+               dbus_set_error_const(error, DBUS_ERROR_FAILED, "an error "
+                                    "occurred getting persistent groups list");
+               return FALSE;
        }
 
        for (ssid = wpa_s->conf->ssid; ssid; ssid = ssid->next)
                if (network_is_persistent_group(ssid))
                        num++;
 
-       paths = os_zalloc(num * sizeof(char *));
+       paths = os_calloc(num, sizeof(char *));
        if (!paths) {
-               return dbus_message_new_error(message, DBUS_ERROR_NO_MEMORY,
-                                             NULL);
+               dbus_set_error_const(error, DBUS_ERROR_NO_MEMORY, "no memory");
+               return FALSE;
        }
 
        /* Loop through configured networks and append object path of each */
@@ -1301,9 +1521,8 @@ DBusMessage * wpas_dbus_getter_persistent_groups(DBusMessage *message,
                        continue;
                paths[i] = os_zalloc(WPAS_DBUS_OBJECT_PATH_MAX);
                if (paths[i] == NULL) {
-                       reply = dbus_message_new_error(message,
-                                                      DBUS_ERROR_NO_MEMORY,
-                                                      NULL);
+                       dbus_set_error_const(error, DBUS_ERROR_NO_MEMORY,
+                                            "no memory");
                        goto out;
                }
                /* Construct the object path for this network. */
@@ -1312,74 +1531,65 @@ DBusMessage * wpas_dbus_getter_persistent_groups(DBusMessage *message,
                            wpa_s->dbus_new_path, ssid->id);
        }
 
-       reply = wpas_dbus_simple_array_property_getter(message,
-                                                      DBUS_TYPE_OBJECT_PATH,
-                                                      paths, num);
+       success = wpas_dbus_simple_array_property_getter(iter,
+                                                        DBUS_TYPE_OBJECT_PATH,
+                                                        paths, num, error);
 
 out:
        while (i)
                os_free(paths[--i]);
        os_free(paths);
-       return reply;
+       return success;
 }
 
 
 /**
  * wpas_dbus_getter_persistent_group_properties - Get options for a persistent
  *     group
- * @message: Pointer to incoming dbus message
- * @net: wpa_supplicant structure for a network interface and
- * wpa_ssid structure for a configured persistent group (internally network)
- * Returns: DBus message with network properties or DBus error on failure
+ * @iter: Pointer to incoming dbus message iter
+ * @error: Location to store error on failure
+ * @user_data: Function specific data
+ * Returns: TRUE on success, FALSE on failure
  *
  * Getter for "Properties" property of a persistent group.
  */
-DBusMessage * wpas_dbus_getter_persistent_group_properties(
-       DBusMessage *message, struct network_handler_args *net)
+dbus_bool_t wpas_dbus_getter_persistent_group_properties(DBusMessageIter *iter,
+                                                        DBusError *error,
+                                                        void *user_data)
 {
-       /*
-        * Leveraging the fact that persistent group object is still
+       struct network_handler_args *net = user_data;
+
+       /* Leveraging the fact that persistent group object is still
         * represented in same manner as network within.
         */
-       return wpas_dbus_getter_network_properties(message, net);
+       return wpas_dbus_getter_network_properties(iter, error, net);
 }
 
 
 /**
  * wpas_dbus_setter_persistent_group_properties - Get options for a persistent
  *     group
- * @message: Pointer to incoming dbus message
- * @net: wpa_supplicant structure for a network interface and
- * wpa_ssid structure for a configured persistent group (internally network)
- * Returns: DBus message with network properties or DBus error on failure
+ * @iter: Pointer to incoming dbus message iter
+ * @error: Location to store error on failure
+ * @user_data: Function specific data
+ * Returns: TRUE on success, FALSE on failure
  *
  * Setter for "Properties" property of a persistent group.
  */
-DBusMessage * wpas_dbus_setter_persistent_group_properties(
-       DBusMessage *message, struct network_handler_args *net)
+dbus_bool_t wpas_dbus_setter_persistent_group_properties(DBusMessageIter *iter,
+                                                        DBusError *error,
+                                                        void *user_data)
 {
+       struct network_handler_args *net = user_data;
        struct wpa_ssid *ssid = net->ssid;
-       DBusMessage *reply = NULL;
-       DBusMessageIter iter, variant_iter;
-
-       dbus_message_iter_init(message, &iter);
-
-       dbus_message_iter_next(&iter);
-       dbus_message_iter_next(&iter);
-
-       dbus_message_iter_recurse(&iter, &variant_iter);
+       DBusMessageIter variant_iter;
 
        /*
         * Leveraging the fact that persistent group object is still
         * represented in same manner as network within.
         */
-       reply = set_network_properties(message, net->wpa_s, ssid,
-                                      &variant_iter);
-       if (reply)
-               wpa_printf(MSG_DEBUG, "dbus control interface couldn't set "
-                          "persistent group properties");
-
-       return reply;
+       dbus_message_iter_recurse(iter, &variant_iter);
+       return set_network_properties(net->wpa_s, ssid, &variant_iter, error);
 }
 
 
@@ -1401,6 +1611,7 @@ DBusMessage * wpas_dbus_handler_add_persistent_group(
        DBusMessageIter iter;
        struct wpa_ssid *ssid = NULL;
        char path_buf[WPAS_DBUS_OBJECT_PATH_MAX], *path = path_buf;
+       DBusError error;
 
        dbus_message_iter_init(message, &iter);
 
@@ -1422,11 +1633,16 @@ DBusMessage * wpas_dbus_handler_add_persistent_group(
 
        wpa_config_set_network_defaults(ssid);
 
-       reply = set_network_properties(message, wpa_s, ssid, &iter);
-       if (reply) {
+       dbus_error_init(&error);
+       if (!set_network_properties(wpa_s, ssid, &iter, &error)) {
                wpa_printf(MSG_DEBUG, "dbus: %s: "
                           "Control interface could not set persistent group "
                           "properties", __func__);
+               reply = wpas_dbus_reply_new_from_error(message, &error,
+                                                      DBUS_ERROR_INVALID_ARGS,
+                                                      "Failed to set network "
+                                                      "properties");
+               dbus_error_free(&error);
                goto err;
        }
 
@@ -1572,31 +1788,35 @@ DBusMessage * wpas_dbus_handler_remove_all_persistent_groups(
  * Group object properties accessor methods
  */
 
-DBusMessage * wpas_dbus_getter_p2p_group_members(DBusMessage *message,
-                                               struct wpa_supplicant *wpa_s)
+dbus_bool_t wpas_dbus_getter_p2p_group_members(DBusMessageIter *iter,
+                                              DBusError *error,
+                                              void *user_data)
 {
-       DBusMessage *reply = NULL;
+       struct wpa_supplicant *wpa_s = user_data;
        struct wpa_ssid *ssid;
        unsigned int num_members;
        char **paths;
        unsigned int i;
        void *next = NULL;
        const u8 *addr;
+       dbus_bool_t success = FALSE;
 
-       /* Ensure we are a GO */
-       if (wpa_s->wpa_state != WPA_COMPLETED)
-               return NULL;
+       /* Verify correct role for this property */
+       if (wpas_get_p2p_role(wpa_s) != WPAS_P2P_ROLE_GO) {
+               return wpas_dbus_simple_array_property_getter(
+                       iter, DBUS_TYPE_OBJECT_PATH, NULL, 0, error);
+       }
 
        ssid = wpa_s->conf->ssid;
        /* At present WPAS P2P_GO mode only applicable for p2p_go */
        if (ssid->mode != WPAS_MODE_P2P_GO &&
            ssid->mode != WPAS_MODE_AP &&
            ssid->mode != WPAS_MODE_P2P_GROUP_FORMATION)
-               return NULL;
+               return FALSE;
 
        num_members = p2p_get_group_num_members(wpa_s->p2p_group);
 
-       paths = os_zalloc(num_members * sizeof(char *));
+       paths = os_calloc(num_members, sizeof(char *));
        if (!paths)
                goto out_of_memory;
 
@@ -1612,108 +1832,190 @@ DBusMessage * wpas_dbus_getter_p2p_group_members(DBusMessage *message,
                i++;
        }
 
-       reply = wpas_dbus_simple_array_property_getter(message,
-                                                      DBUS_TYPE_OBJECT_PATH,
-                                                      paths, num_members);
+       success = wpas_dbus_simple_array_property_getter(iter,
+                                                        DBUS_TYPE_OBJECT_PATH,
+                                                        paths, num_members,
+                                                        error);
 
        for (i = 0; i < num_members; i++)
                os_free(paths[i]);
        os_free(paths);
-       return reply;
+       return success;
 
 out_of_memory:
-       reply = dbus_message_new_error(message, DBUS_ERROR_NO_MEMORY, NULL);
+       dbus_set_error_const(error, DBUS_ERROR_NO_MEMORY, "no memory");
        if (paths) {
                for (i = 0; i < num_members; i++)
                        os_free(paths[i]);
                os_free(paths);
        }
-       return reply;
+       return FALSE;
 }
 
 
-DBusMessage * wpas_dbus_getter_p2p_group_properties(
-       DBusMessage *message, struct wpa_supplicant *wpa_s)
+dbus_bool_t wpas_dbus_getter_p2p_group_ssid(DBusMessageIter *iter,
+                                           DBusError *error, void *user_data)
 {
-       DBusMessage *reply = NULL;
-       DBusMessageIter iter, variant_iter, dict_iter;
-       struct hostapd_data *hapd = wpa_s->ap_iface->bss[0];
-       const struct wpabuf *vendor_ext[MAX_WPS_VENDOR_EXTENSIONS];
-       int num_vendor_ext = 0;
-       int i;
+       struct wpa_supplicant *wpa_s = user_data;
+       if (wpa_s->current_ssid == NULL)
+               return FALSE;
+       return wpas_dbus_simple_array_property_getter(
+               iter, DBUS_TYPE_BYTE, wpa_s->current_ssid->ssid,
+               wpa_s->current_ssid->ssid_len, error);
+}
 
-       if (!hapd) {
-               reply = dbus_message_new_error(message, DBUS_ERROR_FAILED,
-                                              NULL);
-               return reply;
+
+dbus_bool_t wpas_dbus_getter_p2p_group_bssid(DBusMessageIter *iter,
+                                            DBusError *error,
+                                            void *user_data)
+{
+       struct wpa_supplicant *wpa_s = user_data;
+       u8 role = wpas_get_p2p_role(wpa_s);
+       u8 *p_bssid;
+
+       if (role == WPAS_P2P_ROLE_CLIENT) {
+               if (wpa_s->current_ssid == NULL)
+                       return FALSE;
+               p_bssid = wpa_s->current_ssid->bssid;
+       } else {
+               if (wpa_s->ap_iface == NULL)
+                       return FALSE;
+               p_bssid = wpa_s->ap_iface->bss[0]->own_addr;
        }
 
-       if (message == NULL)
-               reply = dbus_message_new(DBUS_MESSAGE_TYPE_SIGNAL);
-       else
-               reply = dbus_message_new_method_return(message);
+       return wpas_dbus_simple_array_property_getter(iter, DBUS_TYPE_BYTE,
+                                                     p_bssid, ETH_ALEN,
+                                                     error);
+}
 
-       if (!reply)
-               goto err_no_mem;
 
-       dbus_message_iter_init_append(reply, &iter);
+dbus_bool_t wpas_dbus_getter_p2p_group_frequency(DBusMessageIter *iter,
+                                                DBusError *error,
+                                                void *user_data)
+{
+       struct wpa_supplicant *wpa_s = user_data;
+       u16 op_freq;
+       u8 role = wpas_get_p2p_role(wpa_s);
+
+       if (role == WPAS_P2P_ROLE_CLIENT) {
+               if (wpa_s->go_params == NULL)
+                       return FALSE;
+               op_freq = wpa_s->go_params->freq;
+       } else {
+               if (wpa_s->ap_iface == NULL)
+                       return FALSE;
+               op_freq = wpa_s->ap_iface->freq;
+       }
+
+       return wpas_dbus_simple_property_getter(iter, DBUS_TYPE_UINT16,
+                                               &op_freq, error);
+}
 
-       if (!dbus_message_iter_open_container(&iter, DBUS_TYPE_VARIANT,
-                                             "a{sv}", &variant_iter) ||
-           !wpa_dbus_dict_open_write(&variant_iter, &dict_iter))
-               goto err_no_mem;
 
-       /* Parse WPS Vendor Extensions sent in Beacon/Probe Response */
-       for (i = 0; i < MAX_WPS_VENDOR_EXTENSIONS; i++) {
-               if (hapd->conf->wps_vendor_ext[i] == NULL)
-                       continue;
-               vendor_ext[num_vendor_ext++] = hapd->conf->wps_vendor_ext[i];
-       }
+dbus_bool_t wpas_dbus_getter_p2p_group_passphrase(DBusMessageIter *iter,
+                                                 DBusError *error,
+                                                 void *user_data)
+{
+       struct wpa_supplicant *wpa_s = user_data;
+       u8 role = wpas_get_p2p_role(wpa_s);
+       char *p_pass = NULL;
+
+       /* Verify correct role for this property */
+       if (role == WPAS_P2P_ROLE_GO) {
+               if (wpa_s->current_ssid == NULL)
+                       return FALSE;
+               p_pass = wpa_s->current_ssid->passphrase;
+       } else
+               p_pass = "";
 
-       if (!wpa_dbus_dict_append_wpabuf_array(&dict_iter,
-                                              "WPSVendorExtensions",
-                                              vendor_ext, num_vendor_ext))
-               goto err_no_mem;
+       return wpas_dbus_simple_property_getter(iter, DBUS_TYPE_STRING,
+                                               &p_pass, error);
 
-       if (!wpa_dbus_dict_close_write(&variant_iter, &dict_iter) ||
-           !dbus_message_iter_close_container(&iter, &variant_iter))
-               goto err_no_mem;
+}
 
-       return reply;
 
-err_no_mem:
-       dbus_message_unref(reply);
-       return dbus_message_new_error(message, DBUS_ERROR_NO_MEMORY, NULL);
+dbus_bool_t wpas_dbus_getter_p2p_group_psk(DBusMessageIter *iter,
+                                          DBusError *error, void *user_data)
+{
+       struct wpa_supplicant *wpa_s = user_data;
+       u8 role = wpas_get_p2p_role(wpa_s);
+       u8 *p_psk = NULL;
+       u8 psk_len = 0;
+
+       /* Verify correct role for this property */
+       if (role == WPAS_P2P_ROLE_CLIENT) {
+               if (wpa_s->current_ssid == NULL)
+                       return FALSE;
+               p_psk = wpa_s->current_ssid->psk;
+               psk_len = 32;
+       }
+
+       return wpas_dbus_simple_array_property_getter(iter, DBUS_TYPE_BYTE,
+                                                     &p_psk, psk_len, error);
 }
 
 
-DBusMessage * wpas_dbus_setter_p2p_group_properties(
-       DBusMessage *message, struct wpa_supplicant *wpa_s)
+dbus_bool_t wpas_dbus_getter_p2p_group_vendor_ext(DBusMessageIter *iter,
+                                                 DBusError *error,
+                                                 void *user_data)
 {
-       DBusMessage *reply = NULL;
-       DBusMessageIter iter, variant_iter;
-       struct wpa_dbus_dict_entry entry = { .type = DBUS_TYPE_STRING };
-       DBusMessageIter iter_dict;
-       unsigned int i;
-       struct hostapd_data *hapd = wpa_s->ap_iface->bss[0];
+       struct wpa_supplicant *wpa_s = user_data;
+       struct hostapd_data *hapd;
+       struct wpabuf *vendor_ext[MAX_WPS_VENDOR_EXTENSIONS];
+       int num_vendor_ext = 0;
+       int i;
 
-       if (!hapd)
-               goto error;
+       /* Verify correct role for this property */
+       if (wpas_get_p2p_role(wpa_s) == WPAS_P2P_ROLE_GO) {
+               if (wpa_s->ap_iface == NULL)
+                       return FALSE;
+               hapd = wpa_s->ap_iface->bss[0];
+
+               /* Parse WPS Vendor Extensions sent in Beacon/Probe Response */
+               for (i = 0; i < MAX_WPS_VENDOR_EXTENSIONS; i++) {
+                       if (hapd->conf->wps_vendor_ext[i] == NULL)
+                               vendor_ext[i] = NULL;
+                       else {
+                               vendor_ext[num_vendor_ext++] =
+                                       hapd->conf->wps_vendor_ext[i];
+                       }
+               }
+       }
 
-       dbus_message_iter_init(message, &iter);
+       /* Return vendor extensions or no data */
+       return wpas_dbus_simple_array_array_property_getter(iter,
+                                                           DBUS_TYPE_BYTE,
+                                                           vendor_ext,
+                                                           num_vendor_ext,
+                                                error);
+}
 
-       dbus_message_iter_next(&iter);
-       dbus_message_iter_next(&iter);
 
-       dbus_message_iter_recurse(&iter, &variant_iter);
+dbus_bool_t wpas_dbus_setter_p2p_group_vendor_ext(DBusMessageIter *iter,
+                                                 DBusError *error,
+                                                 void *user_data)
+{
+       struct wpa_supplicant *wpa_s = user_data;
+       DBusMessageIter variant_iter, iter_dict;
+       struct wpa_dbus_dict_entry entry = { .type = DBUS_TYPE_STRING };
+       unsigned int i;
+       struct hostapd_data *hapd = NULL;
 
-       if (!wpa_dbus_dict_open_read(&variant_iter, &iter_dict))
-               return wpas_dbus_error_invalid_args(message, NULL);
+       if (wpas_get_p2p_role(wpa_s) == WPAS_P2P_ROLE_GO &&
+           wpa_s->ap_iface != NULL)
+               hapd = wpa_s->ap_iface->bss[0];
+       else
+               return FALSE;
+
+       dbus_message_iter_recurse(iter, &variant_iter);
+       if (!wpa_dbus_dict_open_read(&variant_iter, &iter_dict, error))
+               return FALSE;
 
        while (wpa_dbus_dict_has_dict_entry(&iter_dict)) {
                if (!wpa_dbus_dict_get_entry(&iter_dict, &entry)) {
-                       reply = wpas_dbus_error_invalid_args(message, NULL);
-                       break;
+                       dbus_set_error_const(error, DBUS_ERROR_INVALID_ARGS,
+                                            "invalid message format");
+                       return FALSE;
                }
 
                if (os_strcmp(entry.key, "WPSVendorExtensions") == 0) {
@@ -1738,13 +2040,13 @@ DBusMessage * wpas_dbus_setter_p2p_group_properties(
                wpa_dbus_dict_entry_clear(&entry);
        }
 
-       return reply;
+       return TRUE;
 
 error:
-       reply = wpas_dbus_error_invalid_args(message, entry.key);
        wpa_dbus_dict_entry_clear(&entry);
-
-       return reply;
+       dbus_set_error_const(error, DBUS_ERROR_INVALID_ARGS,
+                            "invalid message format");
+       return FALSE;
 }
 
 
@@ -1764,10 +2066,10 @@ DBusMessage * wpas_dbus_handler_p2p_add_service(DBusMessage *message,
 
        dbus_message_iter_init(message, &iter);
 
-       if (!wpa_dbus_dict_open_read(&iter, &iter_dict))
+       if (!wpa_dbus_dict_open_read(&iter, &iter_dict, NULL))
                goto error;
 
-       if (wpa_dbus_dict_has_dict_entry(&iter_dict)) {
+       while (wpa_dbus_dict_has_dict_entry(&iter_dict)) {
                if (!wpa_dbus_dict_get_entry(&iter_dict, &entry))
                        goto error;
 
@@ -1779,23 +2081,30 @@ DBusMessage * wpas_dbus_handler_p2p_add_service(DBusMessage *message,
                                bonjour = 1;
                        else
                                goto error_clear;
-                       wpa_dbus_dict_entry_clear(&entry);
+               } else if (!os_strcmp(entry.key, "version") &&
+                          entry.type == DBUS_TYPE_INT32) {
+                       version = entry.uint32_value;
+               } else if (!os_strcmp(entry.key, "service") &&
+                            (entry.type == DBUS_TYPE_STRING)) {
+                       service = os_strdup(entry.str_value);
+               } else if (!os_strcmp(entry.key, "query")) {
+                       if ((entry.type != DBUS_TYPE_ARRAY) ||
+                           (entry.array_type != DBUS_TYPE_BYTE))
+                               goto error_clear;
+                       query = wpabuf_alloc_copy(
+                               entry.bytearray_value,
+                               entry.array_len);
+               } else if (!os_strcmp(entry.key, "response")) {
+                       if ((entry.type != DBUS_TYPE_ARRAY) ||
+                           (entry.array_type != DBUS_TYPE_BYTE))
+                               goto error_clear;
+                       resp = wpabuf_alloc_copy(entry.bytearray_value,
+                                                entry.array_len);
                }
+               wpa_dbus_dict_entry_clear(&entry);
        }
 
        if (upnp == 1) {
-               while (wpa_dbus_dict_has_dict_entry(&iter_dict)) {
-                       if (!wpa_dbus_dict_get_entry(&iter_dict, &entry))
-                               goto error;
-
-                       if (!os_strcmp(entry.key, "version") &&
-                           entry.type == DBUS_TYPE_INT32)
-                               version = entry.uint32_value;
-                       else if (!os_strcmp(entry.key, "service") &&
-                                entry.type == DBUS_TYPE_STRING)
-                               service = os_strdup(entry.str_value);
-                       wpa_dbus_dict_entry_clear(&entry);
-               }
                if (version <= 0 || service == NULL)
                        goto error;
 
@@ -1803,37 +2112,15 @@ DBusMessage * wpas_dbus_handler_p2p_add_service(DBusMessage *message,
                        goto error;
 
                os_free(service);
+               service = NULL;
        } else if (bonjour == 1) {
-               while (wpa_dbus_dict_has_dict_entry(&iter_dict)) {
-                       if (!wpa_dbus_dict_get_entry(&iter_dict, &entry))
-                               goto error;
-
-                       if (!os_strcmp(entry.key, "query")) {
-                               if ((entry.type != DBUS_TYPE_ARRAY) ||
-                                   (entry.array_type != DBUS_TYPE_BYTE))
-                                       goto error_clear;
-                               query = wpabuf_alloc_copy(
-                                       entry.bytearray_value,
-                                       entry.array_len);
-                       } else if (!os_strcmp(entry.key, "response")) {
-                               if ((entry.type != DBUS_TYPE_ARRAY) ||
-                                   (entry.array_type != DBUS_TYPE_BYTE))
-                                       goto error_clear;
-                               resp = wpabuf_alloc_copy(entry.bytearray_value,
-                                                        entry.array_len);
-                       }
-
-                       wpa_dbus_dict_entry_clear(&entry);
-               }
-
                if (query == NULL || resp == NULL)
                        goto error;
 
-               if (wpas_p2p_service_add_bonjour(wpa_s, query, resp) < 0) {
-                       wpabuf_free(query);
-                       wpabuf_free(resp);
+               if (wpas_p2p_service_add_bonjour(wpa_s, query, resp) < 0)
                        goto error;
-               }
+               query = NULL;
+               resp = NULL;
        } else
                goto error;
 
@@ -1841,6 +2128,9 @@ DBusMessage * wpas_dbus_handler_p2p_add_service(DBusMessage *message,
 error_clear:
        wpa_dbus_dict_entry_clear(&entry);
 error:
+       os_free(service);
+       wpabuf_free(query);
+       wpabuf_free(resp);
        return wpas_dbus_error_invalid_args(message, NULL);
 }
 
@@ -1861,7 +2151,7 @@ DBusMessage * wpas_dbus_handler_p2p_delete_service(
 
        dbus_message_iter_init(message, &iter);
 
-       if (!wpa_dbus_dict_open_read(&iter, &iter_dict))
+       if (!wpa_dbus_dict_open_read(&iter, &iter_dict, NULL))
                goto error;
 
        if (wpa_dbus_dict_has_dict_entry(&iter_dict)) {
@@ -1959,11 +2249,11 @@ DBusMessage * wpas_dbus_handler_p2p_service_sd_req(
        struct wpabuf *tlv = NULL;
        u8 version = 0;
        u64 ref = 0;
-       u8 addr[ETH_ALEN];
+       u8 addr_buf[ETH_ALEN], *addr;
 
        dbus_message_iter_init(message, &iter);
 
-       if (!wpa_dbus_dict_open_read(&iter, &iter_dict))
+       if (!wpa_dbus_dict_open_read(&iter, &iter_dict, NULL))
                goto error;
 
        while (wpa_dbus_dict_has_dict_entry(&iter_dict)) {
@@ -1996,22 +2286,25 @@ DBusMessage * wpas_dbus_handler_p2p_service_sd_req(
                wpa_dbus_dict_entry_clear(&entry);
        }
 
-       if (!peer_object_path ||
-           (parse_peer_object_path(peer_object_path, addr) < 0) ||
-           (p2p_get_peer_info(wpa_s->global->p2p, addr, 0, NULL, 0) < 0))
-               goto error;
+       if (!peer_object_path) {
+               addr = NULL;
+       } else {
+               if (parse_peer_object_path(peer_object_path, addr_buf) < 0 ||
+                   !p2p_peer_known(wpa_s->global->p2p, addr_buf))
+                       goto error;
+
+               addr = addr_buf;
+       }
 
        if (upnp == 1) {
                if (version <= 0 || service == NULL)
                        goto error;
 
-               ref = (unsigned long) wpas_p2p_sd_request_upnp(wpa_s, addr,
-                                                              version,
-                                                              service);
+               ref = wpas_p2p_sd_request_upnp(wpa_s, addr, version, service);
        } else {
                if (tlv == NULL)
                        goto error;
-               ref = (unsigned long)wpas_p2p_sd_request(wpa_s, addr, tlv);
+               ref = wpas_p2p_sd_request(wpa_s, addr, tlv);
                wpabuf_free(tlv);
        }
 
@@ -2052,7 +2345,7 @@ DBusMessage * wpas_dbus_handler_p2p_service_sd_res(
 
        dbus_message_iter_init(message, &iter);
 
-       if (!wpa_dbus_dict_open_read(&iter, &iter_dict))
+       if (!wpa_dbus_dict_open_read(&iter, &iter_dict, NULL))
                goto error;
 
        while (wpa_dbus_dict_has_dict_entry(&iter_dict)) {
@@ -2081,7 +2374,7 @@ DBusMessage * wpas_dbus_handler_p2p_service_sd_res(
        }
        if (!peer_object_path ||
            (parse_peer_object_path(peer_object_path, addr) < 0) ||
-           (p2p_get_peer_info(wpa_s->global->p2p, addr, 0, NULL, 0) < 0))
+           !p2p_peer_known(wpa_s->global->p2p, addr))
                goto error;
 
        if (tlv == NULL)
@@ -2112,7 +2405,7 @@ DBusMessage * wpas_dbus_handler_p2p_service_sd_cancel_req(
        if (req == 0)
                goto error;
 
-       if (!wpas_p2p_sd_cancel_request(wpa_s, (void *)(unsigned long) req))
+       if (!wpas_p2p_sd_cancel_request(wpa_s, req))
                goto error;
 
        return NULL;