On Fri, Aug 29, 2025 at 06:22:44PM +0000, Ajay.Kathat@xxxxxxxxxxxxx wrote: > Fix the following copy overflow warning identified by Smatch static checker. > > drivers/net/wireless/microchip/wilc1000/wlan_cfg.c:184 wilc_wlan_parse_response_frame() > error: '__memcpy()' 'cfg->s[i]->str' copy overflow (512 vs 65537) > > This patch introduces size check before accessing the memory buffer. > The checks are base on the WID type of received data from the firmware. > For WID string configuration, the size limit is determined by the maximum > element size in 'struct wilc_cfg_str_vals'. Therefore, WILC_MAX_CFG_STR_SIZE > macro is added to configure this size. > > Reported-by: Dan Carpenter <dan.carpenter@xxxxxxxxxx> > Closes: https://lore.kernel.org/linux-wireless/aLFbr9Yu9j_TQTey@stanley.mountain > Suggested-by: Dan Carpenter <dan.carpenter@xxxxxxxxxx> > Signed-off-by: Ajay Singh <ajay.kathat@xxxxxxxxxxxxx> > --- Thanks so much! Reviewed-by: Dan Carpenter <dan.carpenter@xxxxxxxxxx> regards, dan carpenter
