Re: [PATCH] dbus: Share BSSID of AP in Credentials signal

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Tue, 2018-07-03 at 16:47 +0530, Saurav Babu wrote:
> wpa_s->current_ssid->bssid is empty when Credentials signal is sent
> whereas cred->mac_addr contains actual MAC Address of AP for which
> Credentials signal is being sent.
> This patch sends cred->mac_addr as BSSID in Credentials signal so
> that
> application is aware about the actual MAC Address of the AP for which
> signal is received

LGTM

Dan

> Signed-off-by: Saurav Babu <saurav.babu@xxxxxxxxxxx>
> ---
>  wpa_supplicant/dbus/dbus_new.c | 5 ++---
>  1 file changed, 2 insertions(+), 3 deletions(-)
> 
> diff --git a/wpa_supplicant/dbus/dbus_new.c
> b/wpa_supplicant/dbus/dbus_new.c
> index e0f16bb..1a4ffc4 100644
> --- a/wpa_supplicant/dbus/dbus_new.c
> +++ b/wpa_supplicant/dbus/dbus_new.c
> @@ -764,10 +764,9 @@ void wpas_dbus_signal_wps_cred(struct
> wpa_supplicant *wpa_s,
>  	if (cred->encr_type & WPS_ENCR_AES)
>  		encr_type[et_num++] = "aes";
>  
> -	if ((wpa_s->current_ssid &&
> -	     !wpa_dbus_dict_append_byte_array(
> +	if (!wpa_dbus_dict_append_byte_array(
>  		     &dict_iter, "BSSID",
> -		     (const char *) wpa_s->current_ssid->bssid,
> ETH_ALEN)) ||
> +		     (const char *) cred->mac_addr, ETH_ALEN) ||
>  	    !wpa_dbus_dict_append_byte_array(&dict_iter, "SSID",
>  					     (const char *) cred-
> >ssid,
>  					     cred->ssid_len) ||

_______________________________________________
Hostap mailing list
Hostap@xxxxxxxxxxxxxxxxxxx
http://lists.infradead.org/mailman/listinfo/hostap



[Index of Archives]     [Linux Wireless]     [Linux Kernel]     [ATH6KL]     [Linux Bluetooth]     [Linux Netdev]     [Kernel Newbies]     [IDE]     [Security]     [Git]     [Netfilter]     [Bugtraq]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Linux ATA RAID]     [Samba]     [Device Mapper]

  Powered by Linux