Return-path: Received: from esa4.microchip.iphmx.com ([68.232.154.123]:24642 "EHLO esa4.microchip.iphmx.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S965017AbeEITTp (ORCPT ); Wed, 9 May 2018 15:19:45 -0400 Date: Thu, 10 May 2018 00:47:31 +0530 From: Ajay Singh To: Claudiu Beznea CC: , , , , , , Subject: Re: [PATCH 15/30] staging: wilc1000: use kmemdup instead of kmalloc in add_network_to_shadow() Message-ID: <20180510004731.64cd2d32@ajaysk-VirtualBox> (sfid-20180509_211948_687179_734AA165) In-Reply-To: References: <1525682614-3824-1-git-send-email-ajay.kathat@microchip.com> <1525682614-3824-16-git-send-email-ajay.kathat@microchip.com> MIME-Version: 1.0 Content-Type: text/plain; charset="US-ASCII" Sender: linux-wireless-owner@vger.kernel.org List-ID: On Wed, 9 May 2018 16:42:59 +0300 Claudiu Beznea wrote: > 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 > > --- > > 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 ? I couldn't find code where 'ies_len' is check to validity of data. Mostly we use NULL check for "ies" pointer for data validity.So in my opinion setting it to zero would be irrelevant. > > > shadow_nw_info->time_scan = jiffies; > > shadow_nw_info->time_scan_cached = jiffies; > > shadow_nw_info->found = 1; > > Regards, Ajay