RE: [PATCH] ASoC: Intel: fix Coccinelle kfree warning

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



> -----Original Message-----
> From: Firo Yang [mailto:firogm@xxxxxxxxx]
> Sent: Sunday, April 26, 2015 11:42 PM
> To: lgirdwood@xxxxxxxxx
> Cc: kernel-janitors@xxxxxxxxxxxxxxx; broonie@xxxxxxxxxx; perex@xxxxxxxx;
> tiwai@xxxxxxx; Jie, Yang; Lu, Han; jarkko.nikula@xxxxxxxxxxxxxxx; Piskorski,
> Pawel; alsa-devel@xxxxxxxxxxxxxxxx; Firo Yang
> Subject: [PATCH] ASoC: Intel: fix Coccinelle kfree warning
> 
> It's not necessary to use kfree() to free memory allocated with devm_kzalloc
> in a probe function and using kfree leads to a double free.
> 
> Signed-off-by: Firo Yang <firogm@xxxxxxxxx>
> ---
>  sound/soc/intel/haswell/sst-haswell-ipc.c | 1 -
>  1 file changed, 1 deletion(-)
> 
> diff --git a/sound/soc/intel/haswell/sst-haswell-ipc.c
> b/sound/soc/intel/haswell/sst-haswell-ipc.c
> index 344a1e9..324eceb 100644
> --- a/sound/soc/intel/haswell/sst-haswell-ipc.c
> +++ b/sound/soc/intel/haswell/sst-haswell-ipc.c
> @@ -2201,7 +2201,6 @@ dma_err:
>  dsp_new_err:
>  	sst_ipc_fini(ipc);
>  ipc_init_err:
> -	kfree(hsw);
 
It has been fixed by Jin Yao 2 weeks ago:
http://git.kernel.org/cgit/linux/kernel/git/broonie/sound.git/commit/?h=for-next&id=ac98b4c015b50b1e452f8d55b612320be7f80825

~Keyon

>  	return ret;
>  }
>  EXPORT_SYMBOL_GPL(sst_hsw_dsp_init);
> --
> 2.1.0

--
To unsubscribe from this list: send the line "unsubscribe kernel-janitors" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html




[Index of Archives]     [Kernel Development]     [Kernel Announce]     [Kernel Newbies]     [Linux Networking Development]     [Share Photos]     [IDE]     [Security]     [Git]     [Netfilter]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Device Mapper]

  Powered by Linux