summaryrefslogtreecommitdiffstats
path: root/sound
diff options
context:
space:
mode:
authorPhilipp Zabel <p.zabel@pengutronix.de>2013-09-25 15:22:01 +0200
committerMark Brown <broonie@linaro.org>2013-09-26 12:01:32 +0100
commit50d4a790e65f5ac91a7b2720a19e80e862b40318 (patch)
tree7b036ab65805090e553758edd1f82851ec306cb9 /sound
parenta8b22c1ccccd2a5368f803afab41b4301832ff10 (diff)
downloadop-kernel-dev-50d4a790e65f5ac91a7b2720a19e80e862b40318.zip
op-kernel-dev-50d4a790e65f5ac91a7b2720a19e80e862b40318.tar.gz
ASoC: imx-sgtl5000: Fix uninitialized pointer use in error path
This patch avoids to dereference the uninitialized data pointer if the error path is entered before devm_kzalloc is called (or if the allocation fails). It fixes the following warning: sound/soc/fsl/imx-sgtl5000.c: In function 'imx_sgtl5000_probe': sound/soc/fsl/imx-sgtl5000.c:175:18: warning: 'data' may be used uninitialized in this function [-Wmaybe-uninitialized] Signed-off-by: Philipp Zabel <p.zabel@pengutronix.de> Signed-off-by: Mark Brown <broonie@linaro.org>
Diffstat (limited to 'sound')
-rw-r--r--sound/soc/fsl/imx-sgtl5000.c4
1 files changed, 2 insertions, 2 deletions
diff --git a/sound/soc/fsl/imx-sgtl5000.c b/sound/soc/fsl/imx-sgtl5000.c
index 52df7d5..ca1be1d 100644
--- a/sound/soc/fsl/imx-sgtl5000.c
+++ b/sound/soc/fsl/imx-sgtl5000.c
@@ -62,7 +62,7 @@ static int imx_sgtl5000_probe(struct platform_device *pdev)
struct device_node *ssi_np, *codec_np;
struct platform_device *ssi_pdev;
struct i2c_client *codec_dev;
- struct imx_sgtl5000_data *data;
+ struct imx_sgtl5000_data *data = NULL;
int int_port, ext_port;
int ret;
@@ -172,7 +172,7 @@ static int imx_sgtl5000_probe(struct platform_device *pdev)
return 0;
fail:
- if (!IS_ERR(data->codec_clk))
+ if (data && !IS_ERR(data->codec_clk))
clk_put(data->codec_clk);
if (ssi_np)
of_node_put(ssi_np);
OpenPOWER on IntegriCloud