Re: [PATCH v2] staging: rtl8723bs: replace kzalloc/memcpy with kmemdup

[Greg KH <gregkh@linuxfoundation.org>]

On Mon, Mar 09, 2026 at 10:21:50PM +0800, Lin YuChen wrote:
> Replace the open-coded kzalloc and memcpy combination with kmemdup()
> to make the code more concise and idiomatic.
> 
> Signed-off-by: Lin YuChen <starpt.official@gmail.com>
> ---
> v2:
>  - Use real name in Signed-off-by line as requested by Ethan Tidmore.
>  - Fix a minor whitespace issue in the error return path.
>  drivers/staging/rtl8723bs/os_dep/ioctl_cfg80211.c | 6 ++----
>  1 file changed, 2 insertions(+), 4 deletions(-)
> 
> diff --git a/drivers/staging/rtl8723bs/os_dep/ioctl_cfg80211.c b/drivers/staging/rtl8723bs/os_dep/ioctl_cfg80211.c
> index 7cb0c6f22bf3..3db6417a7de1 100644
> --- a/drivers/staging/rtl8723bs/os_dep/ioctl_cfg80211.c
> +++ b/drivers/staging/rtl8723bs/os_dep/ioctl_cfg80211.c
> @@ -1430,14 +1430,12 @@ static int rtw_cfg80211_set_wpa_ie(struct adapter *padapter, u8 *pie, size_t iel
>  		goto exit;
>  	}
>  
> -	buf = kzalloc(ielen, GFP_KERNEL);
> +	buf = kmemdup(pie, ielen, GFP_KERNEL);
>  	if (!buf) {
> -		ret =  -ENOMEM;
> +		ret = -ENOMEM;

You also changed the spacing here, while normally ok, it's not good for
a first patch.  Please only do one logical-thing-per-patch.

thanks,

greg k-h