commit a6d9cef30eb11b2de8cbfed9065e3dc5b1f829a8
author Dan Carpenter <dan.carpenter@oracle.com> Tue Feb 19 15:04:27 2019 +0300
committer Mark Brown <broonie@kernel.org> Tue Feb 19 15:23:25 2019 +0000
tree af6fcdf72ee68f85e0356ade2b2326f1812b8ce4
parent f89aea0f132142b29dc0c8cf4d445bd12db7b1a6

ASoC: dapm: Potential small memory leak in dapm_cnew_widget()

We should free "w" on the error path.

Fixes: 199ed3e81c49 ("ASoC: dapm: fix use-after-free issue with dailink sname")
Signed-off-by: Dan Carpenter <dan.carpenter@oracle.com>
Signed-off-by: Mark Brown <broonie@kernel.org>

sound/soc/soc-dapm.c

diff --git a/sound/soc/soc-dapm.c b/sound/soc/soc-dapm.c
index dea6fc2..1ec06ef 100644
--- a/sound/soc/soc-dapm.c
+++ b/sound/soc/soc-dapm.c
@@ -332,8 +332,10 @@ static inline struct snd_soc_dapm_widget *dapm_cnew_widget(
 	 */
 	if (_widget->sname) {
 		w->sname = kstrdup_const(_widget->sname, GFP_KERNEL);
-		if (!w->sname)
+		if (!w->sname) {
+			kfree(w);
 			return NULL;
+		}
 	}
 	return w;
 }