diff options
| author | Gustavo A. R. Silva <gustavo@embeddedor.com> | 2019-01-15 18:57:23 +0100 |
|---|---|---|
| committer | Mark Brown <broonie@kernel.org> | 2019-01-15 20:03:49 +0100 |
| commit | 060d0bf491874daece47053c4e1fb0489eb867d2 (patch) | |
| tree | 7c0b90d23cafd6c32e3caf6c2dc0e708caa3f2f5 /sound | |
| parent | ASoC: dapm: change snprintf to scnprintf for possible overflow (diff) | |
| download | linux-060d0bf491874daece47053c4e1fb0489eb867d2.tar.xz linux-060d0bf491874daece47053c4e1fb0489eb867d2.zip | |
ASoC: rt5514-spi: Fix potential NULL pointer dereference
There is a potential NULL pointer dereference in case devm_kzalloc()
fails and returns NULL.
Fix this by adding a NULL check on rt5514_dsp.
This issue was detected with the help of Coccinelle.
Fixes: 6eebf35b0e4a ("ASoC: rt5514: add rt5514 SPI driver")
Cc: stable@vger.kernel.org
Signed-off-by: Gustavo A. R. Silva <gustavo@embeddedor.com>
Signed-off-by: Mark Brown <broonie@kernel.org>
Diffstat (limited to 'sound')
| -rw-r--r-- | sound/soc/codecs/rt5514-spi.c | 2 |
1 files changed, 2 insertions, 0 deletions
diff --git a/sound/soc/codecs/rt5514-spi.c b/sound/soc/codecs/rt5514-spi.c index 4d46f4567c3a..bec2eefa8b0f 100644 --- a/sound/soc/codecs/rt5514-spi.c +++ b/sound/soc/codecs/rt5514-spi.c @@ -280,6 +280,8 @@ static int rt5514_spi_pcm_probe(struct snd_soc_component *component) rt5514_dsp = devm_kzalloc(component->dev, sizeof(*rt5514_dsp), GFP_KERNEL); + if (!rt5514_dsp) + return -ENOMEM; rt5514_dsp->dev = &rt5514_spi->dev; mutex_init(&rt5514_dsp->dma_lock); |