[alsa-devel] ASoC: rt5514: add rt5514 SPI driver

Dan Carpenter dan.carpenter at oracle.com
Thu Jun 16 12:49:25 CEST 2016


Hello Oder Chiou,

This is a semi-automatic email about new static checker warnings.

The patch 6eebf35b0e4a: "ASoC: rt5514: add rt5514 SPI driver" from 
Jun 6, 2016, leads to the following Smatch complaint:

sound/soc/codecs/rt5514-spi.c:87 rt5514_spi_copy_work()
	 warn: variable dereferenced before check 'rt5514_dsp->substream' (see line 83)

sound/soc/codecs/rt5514-spi.c
    82			container_of(work, struct rt5514_dsp, copy_work.work);
    83		struct snd_pcm_runtime *runtime = rt5514_dsp->substream->runtime;
                                                  ^^^^^^^^^^^^^^^^^^^^^^^
Dereference.

    84		size_t period_bytes, truncated_bytes = 0;
    85	
    86		mutex_lock(&rt5514_dsp->dma_lock);
    87		if (!rt5514_dsp->substream) {
                     ^^^^^^^^^^^^^^^^^^^^^
Checked too late.

    88			dev_err(rt5514_dsp->dev, "No pcm substream\n");
    89			goto done;

regards,
dan carpenter


More information about the Alsa-devel mailing list