On 07.05.2018 11:43, Ajay Singh wrote: > Use kmemdup instead of kmalloc & memcpy in add_network_to_shadow(). > > Signed-off-by: Ajay Singh <ajay.kathat@xxxxxxxxxxxxx> > --- > drivers/staging/wilc1000/wilc_wfi_cfgoperations.c | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) > > diff --git a/drivers/staging/wilc1000/wilc_wfi_cfgoperations.c b/drivers/staging/wilc1000/wilc_wfi_cfgoperations.c > index 0ae2065..ca221f1 100644 > --- a/drivers/staging/wilc1000/wilc_wfi_cfgoperations.c > +++ b/drivers/staging/wilc1000/wilc_wfi_cfgoperations.c > @@ -331,8 +331,8 @@ static void add_network_to_shadow(struct network_info *nw_info, > shadow_nw_info->tsf_hi = nw_info->tsf_hi; > if (ap_found != -1) > kfree(shadow_nw_info->ies); > - shadow_nw_info->ies = kmalloc(nw_info->ies_len, GFP_KERNEL); > - memcpy(shadow_nw_info->ies, nw_info->ies, nw_info->ies_len); > + shadow_nw_info->ies = kmemdup(nw_info->ies, nw_info->ies_len, > + GFP_KERNEL); Maybe, in case of NULL, you will want to set ies_len = 0 ? > shadow_nw_info->time_scan = jiffies; > shadow_nw_info->time_scan_cached = jiffies; > shadow_nw_info->found = 1; >
