On 2020/7/16 21:29, Jerome Brunet wrote:
On Thu 16 Jul 2020 at 15:25, Jing Xiangfeng jingxiangfeng@huawei.com wrote:
axg_card_add_tdm_loopback() misses to call kfree() in an error path. Add the missed function call to fix it.
Fixes: c84836d7f650 ("ASoC: meson: axg-card: use modern dai_link style") Signed-off-by: Jing Xiangfeng jingxiangfeng@huawei.com
Thanks for fixing this. Maybe it would be better to use the devm_ variant for the name instead ?
Ok, I'll send a v2 with this change.
Thanks for your review.
sound/soc/meson/axg-card.c | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-)
diff --git a/sound/soc/meson/axg-card.c b/sound/soc/meson/axg-card.c index 89f7f64747cd..6eac22ba8b99 100644 --- a/sound/soc/meson/axg-card.c +++ b/sound/soc/meson/axg-card.c @@ -121,8 +121,10 @@ static int axg_card_add_tdm_loopback(struct snd_soc_card *card, return -ENOMEM;
dlc = devm_kzalloc(card->dev, 2 * sizeof(*dlc), GFP_KERNEL);
- if (!dlc)
if (!dlc) {
kfree(lb->name);return -ENOMEM;
}
lb->cpus = &dlc[0]; lb->codecs = &dlc[1];
.