[PATCH] dbus: emit property changed events when adding/removing BSSes

Dan Williams dcbw
Thu Dec 2 08:35:12 PST 2010


On Fri, 2010-11-19 at 14:56 -0600, Dan Williams wrote:
> The supplicant was not emitting property changed events when the BSSs
> property changed.

Ping?  Any particular objection to this patch?

Dan

> Signed-off-by: Dan Williams <dcbw at redhat.com>
> 
> ---
> This is a candidate for 0.7 as well; should apply cleanly.
> 
> diff --git a/wpa_supplicant/dbus/dbus_new.c b/wpa_supplicant/dbus/dbus_new.c
> index bdfbbac..8bd42c5 100644
> --- a/wpa_supplicant/dbus/dbus_new.c
> +++ b/wpa_supplicant/dbus/dbus_new.c
> @@ -691,6 +691,11 @@ void wpas_dbus_signal_prop_changed(struct wpa_supplicant *wpa_s,
>  			wpas_dbus_getter_current_network;
>  		prop = "CurrentNetwork";
>  		break;
> +	case WPAS_DBUS_PROP_BSSS:
> +		getter = (WPADBusPropertyAccessor)
> +			wpas_dbus_getter_bsss;
> +		prop = "BSSs";
> +		break;
>  	default:
>  		wpa_printf(MSG_ERROR, "dbus: %s: Unknown Property value %d",
>  			   __func__, property);
> @@ -1199,6 +1204,7 @@ int wpas_dbus_unregister_bss(struct wpa_supplicant *wpa_s,
>  	}
>  
>  	wpas_dbus_signal_bss_removed(wpa_s, bss_obj_path);
> +	wpas_dbus_signal_prop_changed (wpa_s, WPAS_DBUS_PROP_BSSS);
>  
>  	return 0;
>  }
> @@ -1263,6 +1269,7 @@ int wpas_dbus_register_bss(struct wpa_supplicant *wpa_s,
>  	}
>  
>  	wpas_dbus_signal_bss_added(wpa_s, bss_obj_path);
> +	wpas_dbus_signal_prop_changed (wpa_s, WPAS_DBUS_PROP_BSSS);
>  
>  	return 0;
>  
> diff --git a/wpa_supplicant/dbus/dbus_new.h b/wpa_supplicant/dbus/dbus_new.h
> index 80ea98c..9cdefcb 100644
> --- a/wpa_supplicant/dbus/dbus_new.h
> +++ b/wpa_supplicant/dbus/dbus_new.h
> @@ -30,6 +30,7 @@ enum wpas_dbus_prop {
>  	WPAS_DBUS_PROP_STATE,
>  	WPAS_DBUS_PROP_CURRENT_BSS,
>  	WPAS_DBUS_PROP_CURRENT_NETWORK,
> +	WPAS_DBUS_PROP_BSSS,
>  };
>  
>  enum wpas_dbus_bss_prop {
> 
> _______________________________________________
> HostAP mailing list
> HostAP at lists.shmoo.com
> http://lists.shmoo.com/mailman/listinfo/hostap





More information about the Hostap mailing list