Re: memcpy: detected field-spanning write (size 101) of single field "ext_scan->tlv_buffer" at drivers/net/wireless/marvell/mwifiex/scan.c:2251 (size 1)

[Ahelenia Ziemiańska]

On Mon, Jan 29, 2024 at 04:55:26PM +0300, Dmitry Antipov wrote:
> On 1/26/24 22:47, Ahelenia Ziemiańska wrote:
> > [   41.314634] memcpy: detected field-spanning write (size 101) of single field "ext_scan->tlv_buffer" at drivers/net/wireless/marvell/mwifiex/scan.c:2251 (size 1)
> Short answer: if your device works as expected, most likely you can ignore this.
It does.

> if you're brave
> enough, you can help the maintainers by trying this:
> --- linux-6.6.11/drivers/net/wireless/marvell/mwifiex/fw.h	2024-01-10 19:17:02.000000000 +0300
> +++ linux-6.6.11/drivers/net/wireless/marvell/mwifiex/fw.h	2024-01-29 14:21:55.574280719 +0300
> @@ -1586,7 +1586,7 @@
> 
>  struct host_cmd_ds_802_11_scan_ext {
>  	u32   reserved;
> -	u8    tlv_buffer[1];
> +	u8    tlv_buffer[];
>  } __packed;
> 
>  struct mwifiex_ie_types_bss_mode {

Tested-by: Ahelenia Ziemiańska <nabijaczleweli@xxxxxxxxxxxxxxxxxx>

That works and makes the warning go away for me on 6.6.11 ‒
now all mwifiex-related log is confined to
  mwifiex_sdio mmc2:0001:1: firmware: direct-loading firmware mrvl/sd8897_uapsta.bin
  mwifiex_sdio mmc2:0001:1: WLAN is not the winner! Skip FW dnld
  mwifiex_sdio mmc2:0001:1: WLAN FW is active
  mwifiex_sdio mmc2:0001:1: Bad HW revision detected, disabling deep sleep
  mwifiex_sdio mmc2:0001:1: info: MWIFIEX VERSION: mwifiex 1.0 (15.68.19.p54) 
  mwifiex_sdio mmc2:0001:1: driver_version = mwifiex 1.0 (15.68.19.p54) 
  mwifiex_sdio mmc2:0001:1: info: trying to associate to bssid de:0d:17:ad:84:d9
  mwifiex_sdio mmc2:0001:1: info: associated to bssid de:0d:17:ad:84:d9 successfully
  mwifiex_sdio mmc2:0001:1: event: unknown event id: 0x79

Thanks!

Attachment: signature.asc
Description: PGP signature