asoc: free backend_name memory when error happens
Free backend_name memory when error happens. Change-Id: Ia54e5483f061ec015cc717e0b408804df58af7f0 Signed-off-by: Meng Wang <quic_mengw@quicinc.com>
This commit is contained in:

committed by
Gerrit - the friendly Code Review server

parent
c2adcdf8a0
commit
17e29ced12
@@ -1102,16 +1102,20 @@ int msm_common_dai_link_init(struct snd_soc_pcm_runtime *rtd)
|
|||||||
}
|
}
|
||||||
|
|
||||||
pdata = devm_kzalloc(dev, sizeof(struct chmap_pdata), GFP_KERNEL);
|
pdata = devm_kzalloc(dev, sizeof(struct chmap_pdata), GFP_KERNEL);
|
||||||
if (!pdata)
|
if (!pdata) {
|
||||||
return -ENOMEM;
|
ret = -ENOMEM;
|
||||||
|
goto free_backend;
|
||||||
|
}
|
||||||
|
|
||||||
if ((!strncmp(backend_name, "SLIM", strlen("SLIM"))) ||
|
if ((!strncmp(backend_name, "SLIM", strlen("SLIM"))) ||
|
||||||
(!strncmp(backend_name, "CODEC_DMA", strlen("CODEC_DMA")))) {
|
(!strncmp(backend_name, "CODEC_DMA", strlen("CODEC_DMA")))) {
|
||||||
ctl_len = strlen(dai_link->stream_name) + 1 +
|
ctl_len = strlen(dai_link->stream_name) + 1 +
|
||||||
strlen(mixer_ctl_name) + 1;
|
strlen(mixer_ctl_name) + 1;
|
||||||
mixer_str = kzalloc(ctl_len, GFP_KERNEL);
|
mixer_str = kzalloc(ctl_len, GFP_KERNEL);
|
||||||
if (!mixer_str)
|
if (!mixer_str) {
|
||||||
return -ENOMEM;
|
ret = -ENOMEM;
|
||||||
|
goto free_backend;
|
||||||
|
}
|
||||||
|
|
||||||
snprintf(mixer_str, ctl_len, "%s %s", dai_link->stream_name,
|
snprintf(mixer_str, ctl_len, "%s %s", dai_link->stream_name,
|
||||||
mixer_ctl_name);
|
mixer_ctl_name);
|
||||||
@@ -1149,14 +1153,16 @@ int msm_common_dai_link_init(struct snd_soc_pcm_runtime *rtd)
|
|||||||
}
|
}
|
||||||
|
|
||||||
free_mixer_str:
|
free_mixer_str:
|
||||||
if (backend_name) {
|
|
||||||
kfree(backend_name);
|
|
||||||
backend_name = NULL;
|
|
||||||
}
|
|
||||||
if (mixer_str) {
|
if (mixer_str) {
|
||||||
kfree(mixer_str);
|
kfree(mixer_str);
|
||||||
mixer_str = NULL;
|
mixer_str = NULL;
|
||||||
}
|
}
|
||||||
|
|
||||||
|
free_backend:
|
||||||
|
if (backend_name) {
|
||||||
|
kfree(backend_name);
|
||||||
|
backend_name = NULL;
|
||||||
|
}
|
||||||
|
|
||||||
return ret;
|
return ret;
|
||||||
}
|
}
|
||||||
|
Reference in New Issue
Block a user