Messages in this thread | | | Date | Thu, 16 Apr 2015 17:05:33 +0300 | From | Jarkko Nikula <> | Subject | Re: [PATCH -next] ASoC: Intel: sst_byt: remove kfree for memory allocated with devm_kzalloc |
| |
Hi
On 04/16/2015 04:46 PM, weiyj_lk@163.com wrote: > From: Wei Yongjun <yongjun_wei@trendmicro.com.cn> > > It's not necessary to free memory allocated with devm_kzalloc > and using kfree leads to a double free. > > Signed-off-by: Wei Yongjun <yongjun_wei@trendmicro.com.cn> > --- > sound/soc/intel/baytrail/sst-baytrail-ipc.c | 1 - > 1 file changed, 1 deletion(-) > > diff --git a/sound/soc/intel/baytrail/sst-baytrail-ipc.c b/sound/soc/intel/baytrail/sst-baytrail-ipc.c > index 1efb33b..a839dbf 100644 > --- a/sound/soc/intel/baytrail/sst-baytrail-ipc.c > +++ b/sound/soc/intel/baytrail/sst-baytrail-ipc.c > @@ -759,7 +759,6 @@ fw_err: > dsp_new_err: > sst_ipc_fini(ipc); > ipc_init_err: > - kfree(byt); > > return err; > } > Above kfree(byt) appears to be added by accident in the commit 48cec59b6f38 ("ASoC: Intel: Use the generic IPC/mailbox APIs in Baytrail").
Acked-by: Jarkko Nikula <jarkko.nikula@linux.intel.com>
| |