diff options
author | Kamil Duljas <kamil.duljas@gmail.com> | 2023-11-16 15:51:50 +0300 |
---|---|---|
committer | Mark Brown <broonie@kernel.org> | 2023-11-17 15:47:49 +0300 |
commit | d5c65be34df73fa01ed05611aafb73b440d89e29 (patch) | |
tree | 08979c021d8d46b25d05a46901d6b9b9eb582b76 /sound/soc/intel/skylake/skl-pcm.c | |
parent | 37e6fd0cebf0b9f71afb38fd95b10408799d1f0b (diff) | |
download | linux-d5c65be34df73fa01ed05611aafb73b440d89e29.tar.xz |
ASoC: Intel: Skylake: Fix mem leak in few functions
The resources should be freed when function return error.
Signed-off-by: Kamil Duljas <kamil.duljas@gmail.com>
Reviewed-by: Amadeusz Sławiński <amadeuszx.slawinski@linux.intel.com>
Link: https://lore.kernel.org/r/20231116125150.1436-1-kamil.duljas@gmail.com
Signed-off-by: Mark Brown <broonie@kernel.org>
Diffstat (limited to 'sound/soc/intel/skylake/skl-pcm.c')
-rw-r--r-- | sound/soc/intel/skylake/skl-pcm.c | 4 |
1 files changed, 3 insertions, 1 deletions
diff --git a/sound/soc/intel/skylake/skl-pcm.c b/sound/soc/intel/skylake/skl-pcm.c index d0c02e8a6785..18866bc415a5 100644 --- a/sound/soc/intel/skylake/skl-pcm.c +++ b/sound/soc/intel/skylake/skl-pcm.c @@ -240,8 +240,10 @@ static int skl_pcm_open(struct snd_pcm_substream *substream, snd_pcm_set_sync(substream); mconfig = skl_tplg_fe_get_cpr_module(dai, substream->stream); - if (!mconfig) + if (!mconfig) { + kfree(dma_params); return -EINVAL; + } skl_tplg_d0i3_get(skl, mconfig->d0i3_caps); |