ASoC: simple-card-utils: test memory allocation
authorKuninori Morimoto <kuninori.morimoto.gx@renesas.com>
Wed, 20 Mar 2019 04:57:02 +0000 (13:57 +0900)
committerMark Brown <broonie@kernel.org>
Thu, 21 Mar 2019 14:58:01 +0000 (14:58 +0000)
li->conf will be 0 if it was not DPCM case.
Then, 1) we shouldn't call devm_kcalloc() with size 0,
2) we need NULL pointer check if li->conf was not 0.
This patch fixed above issues.
Special thanks to Pierre-Louis Bossart

Reported-by: Pierre-Louis Bossart <pierre-louis.bossart@linux.intel.com>
Signed-off-by: Kuninori Morimoto <kuninori.morimoto.gx@renesas.com>
Signed-off-by: Mark Brown <broonie@kernel.org>
sound/soc/generic/simple-card-utils.c

index a794f76..4ed6834 100644 (file)
@@ -495,16 +495,21 @@ int asoc_simple_init_priv(struct asoc_simple_priv *priv,
        struct snd_soc_dai_link *dai_link;
        struct simple_dai_props *dai_props;
        struct asoc_simple_dai *dais;
-       struct snd_soc_codec_conf *cconf;
+       struct snd_soc_codec_conf *cconf = NULL;
        int i;
 
        dai_props = devm_kcalloc(dev, li->link, sizeof(*dai_props), GFP_KERNEL);
        dai_link  = devm_kcalloc(dev, li->link, sizeof(*dai_link),  GFP_KERNEL);
        dais      = devm_kcalloc(dev, li->dais, sizeof(*dais),      GFP_KERNEL);
-       cconf     = devm_kcalloc(dev, li->conf, sizeof(*cconf),     GFP_KERNEL);
        if (!dai_props || !dai_link || !dais)
                return -ENOMEM;
 
+       if (li->conf) {
+               cconf = devm_kcalloc(dev, li->conf, sizeof(*cconf), GFP_KERNEL);
+               if (!cconf)
+                       return -ENOMEM;
+       }
+
        /*
         * Use snd_soc_dai_link_component instead of legacy style
         * It is codec only. but cpu/platform will be supported in the future.