Johannes Berg <johannes@xxxxxxxxxxxxxxxx> writes:
> From: Johannes Berg <johannes.berg@xxxxxxxxx>
>
> Use "u8 bssid_data[]" with an appropriate cast. No binary
> changes.
>
> Signed-off-by: Johannes Berg <johannes.berg@xxxxxxxxx>
> ---
> drivers/net/wireless/rndis_wlan.c | 5 +++--
> 1 file changed, 3 insertions(+), 2 deletions(-)
>
> diff --git a/drivers/net/wireless/rndis_wlan.c b/drivers/net/wireless/rndis_wlan.c
> index 325933217b41..82a7458e01ae 100644
> --- a/drivers/net/wireless/rndis_wlan.c
> +++ b/drivers/net/wireless/rndis_wlan.c
> @@ -251,7 +251,7 @@ struct ndis_80211_bssid_ex {
>
> struct ndis_80211_bssid_list_ex {
> __le32 num_items;
> - struct ndis_80211_bssid_ex bssid[];
> + u8 bssid_data[];
> } __packed;
>
> struct ndis_80211_fixed_ies {
> @@ -2084,7 +2084,8 @@ static int rndis_check_bssid_list(struct usbnet *usbdev, u8 *match_bssid,
> netdev_dbg(usbdev->net, "%s(): buflen: %d\n", __func__, len);
>
> bssid_len = 0;
> - bssid = next_bssid_list_item(bssid_list->bssid, &bssid_len, buf, len);
> + bssid = next_bssid_list_item((void *)bssid_list->bssid_data,
> + &bssid_len, buf, len);
Same comment as in patch 2, I just feel the code gets worse because of a
compiler warning.
--
https://patchwork.kernel.org/project/linux-wireless/list/
https://wireless.wiki.kernel.org/en/developers/documentation/submittingpatches
