[PATCH] WPS: Fix new dbus API
Dan Williams
dcbw
Tue Nov 23 18:42:52 PST 2010
On Tue, 2010-11-23 at 10:42 +0200, Tomasz Bursztyka wrote:
> Start() method's optional Bssid argument needs a byte array, not an array of
> arrays.
Good catch...
Dan
> ---
> wpa_supplicant/dbus/dbus_new_handlers_wps.c | 2 +-
> 1 files changed, 1 insertions(+), 1 deletions(-)
>
> diff --git a/wpa_supplicant/dbus/dbus_new_handlers_wps.c b/wpa_supplicant/dbus/dbus_new_handlers_wps.c
> index 05aed5e..c118d73 100644
> --- a/wpa_supplicant/dbus/dbus_new_handlers_wps.c
> +++ b/wpa_supplicant/dbus/dbus_new_handlers_wps.c
> @@ -107,7 +107,7 @@ static int wpas_dbus_handler_wps_bssid(DBusMessage *message,
> dbus_message_iter_recurse(entry_iter, &variant_iter);
> if (dbus_message_iter_get_arg_type(&variant_iter) != DBUS_TYPE_ARRAY ||
> dbus_message_iter_get_element_type(&variant_iter) !=
> - DBUS_TYPE_ARRAY) {
> + DBUS_TYPE_BYTE) {
> wpa_printf(MSG_DEBUG, "dbus: WPS.Start - Wrong Bssid type, "
> "byte array required");
> *reply = wpas_dbus_error_invalid_args(
More information about the Hostap
mailing list