On Tue, Jul 03, 2018 at 04:47:15PM +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. wpa_s->current_ssid->bssid looks incorrect here, but cred->mac_addr is no better since it is not the MAC address of the AP, it is the MAC address of the WPS Enrollee, i.e., the receives of this Credential. When wpa_supplicant/station is the Enrollee, this address is the MAC address of the local station, not the BSSID of the AP that happens to be forwarding that WPS credential to the station. > 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 That is not BSSID, so that value should not be exposed in a field called BSSID. What would this BSSID field be used for in the first place? wpa_s->bssid would likely contain the BSSID value, but I don't really see much, if any, value in exposing it externally here.. -- Jouni Malinen PGP id EFC895FA _______________________________________________ Hostap mailing list Hostap@xxxxxxxxxxxxxxxxxxx http://lists.infradead.org/mailman/listinfo/hostap
