forked from luck/tmp_suning_uos_patched
ASoC: wm_adsp: fix possible memory leak in wm_adsp_load_coeff()
'file' is malloced in wm_adsp_load_coeff() and should be freed before leaving from the error handling cases, otherwise it will cause memory leak. Signed-off-by: Wei Yongjun <yongjun_wei@trendmicro.com.cn> Signed-off-by: Mark Brown <broonie@opensource.wolfsonmicro.com>
This commit is contained in:
parent
f6161aa153
commit
f4b828128a
@ -828,7 +828,8 @@ static int wm_adsp_load_coeff(struct wm_adsp *dsp)
|
||||
&buf_list);
|
||||
if (!buf) {
|
||||
adsp_err(dsp, "Out of memory\n");
|
||||
return -ENOMEM;
|
||||
ret = -ENOMEM;
|
||||
goto out_fw;
|
||||
}
|
||||
|
||||
adsp_dbg(dsp, "%s.%d: Writing %d bytes at %x\n",
|
||||
@ -865,7 +866,7 @@ static int wm_adsp_load_coeff(struct wm_adsp *dsp)
|
||||
wm_adsp_buf_free(&buf_list);
|
||||
out:
|
||||
kfree(file);
|
||||
return 0;
|
||||
return ret;
|
||||
}
|
||||
|
||||
int wm_adsp1_init(struct wm_adsp *adsp)
|
||||
|
Loading…
Reference in New Issue
Block a user