[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