From: Xiubo Li Date: Thu, 16 Jan 2014 08:08:04 +0000 (+0800) Subject: ASoC: core: Fix possible NULL pointer dereference of pcm->config X-Git-Tag: v3.14-rc1~16^2~10^2~1^2 X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=ec4f2857cd41b7c5af6350e7ab2a8fe1874dae59;p=profile%2Fivi%2Fkernel-x86-ivi.git ASoC: core: Fix possible NULL pointer dereference of pcm->config Since the soc generic dmaengine pcm driver allows using the defualt settings, so the pcm->config maybe NULL. Signed-off-by: Xiubo Li Acked-by: Lars-Peter Clausen Signed-off-by: Mark Brown --- diff --git a/sound/soc/soc-generic-dmaengine-pcm.c b/sound/soc/soc-generic-dmaengine-pcm.c index 560a778..5bace12 100644 --- a/sound/soc/soc-generic-dmaengine-pcm.c +++ b/sound/soc/soc-generic-dmaengine-pcm.c @@ -176,17 +176,20 @@ static struct dma_chan *dmaengine_pcm_compat_request_channel( { struct dmaengine_pcm *pcm = soc_platform_to_pcm(rtd->platform); struct snd_dmaengine_dai_dma_data *dma_data; + dma_filter_fn fn = NULL; dma_data = snd_soc_dai_get_dma_data(rtd->cpu_dai, substream); if ((pcm->flags & SND_DMAENGINE_PCM_FLAG_HALF_DUPLEX) && pcm->chan[0]) return pcm->chan[0]; - if (pcm->config->compat_request_channel) + if (pcm->config && pcm->config->compat_request_channel) return pcm->config->compat_request_channel(rtd, substream); - return snd_dmaengine_pcm_request_channel(pcm->config->compat_filter_fn, - dma_data->filter_data); + if (pcm->config) + fn = pcm->config->compat_filter_fn; + + return snd_dmaengine_pcm_request_channel(fn, dma_data->filter_data); } static bool dmaengine_pcm_can_report_residue(struct dma_chan *chan)