[PATCH] WPS: Fix new dbus API

Dan Williams dcbw at redhat.com
Tue Nov 23 21:42:52 EST 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