[alsa-devel] [PATCH] Revert "wcd9335: fix a incorrect use of kstrndup()"

Gen Zhang blackgod016574 at gmail.com
Wed Jun 12 15:34:53 CEST 2019


On Wed, Jun 12, 2019 at 02:30:40PM +0100, Ben Hutchings wrote:
> This reverts commit a54988113985ca22e414e132054f234fc8a92604.
> The strings being duplicated are not fixed-length, so kstrndup()
> is the correct function to use.
> 
> Signed-off-by: Ben Hutchings <ben.hutchings at codethink.co.uk>
> ---
>  sound/soc/codecs/wcd9335.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/sound/soc/codecs/wcd9335.c b/sound/soc/codecs/wcd9335.c
> index 85737fe54474..a04a7cedd99d 100644
> --- a/sound/soc/codecs/wcd9335.c
> +++ b/sound/soc/codecs/wcd9335.c
> @@ -2734,7 +2734,7 @@ static int wcd9335_codec_enable_dec(struct snd_soc_dapm_widget *w,
>  	char *dec;
>  	u8 hpf_coff_freq;
>  
> -	widget_name = kmemdup_nul(w->name, 15, GFP_KERNEL);
> +	widget_name = kstrndup(w->name, 15, GFP_KERNEL);
>  	if (!widget_name)
>  		return -ENOMEM;
>  
> -- 
> Ben Hutchings, Software Developer                         Codethink Ltd
> https://www.codethink.co.uk/                 Dale House, 35 Dale Street
>                                      Manchester, M1 2HF, United Kingdom
> 
OMG! There is someone following what I was doing these days...
Kind of peculiar


More information about the Alsa-devel mailing list