diff options
| author | Zidan Wang <zidan.wang@freescale.com> | 2015-12-01 16:14:26 +0800 |
|---|---|---|
| committer | Leonard Crestez <leonard.crestez@nxp.com> | 2018-08-24 12:20:42 +0300 |
| commit | 2219842387339fac005839705154a22facd3eee8 (patch) | |
| tree | 7784fccc55ede373f0604f716a4dc513779fc018 /drivers/mfd | |
| parent | 235f3fb59be270cb5f76e91ecc3831bd29657a1d (diff) | |
MLK-11933 mfd: mxc-hdmi-core: null check substream before using it
null check substream before using it. Reported by coverity.
Signed-off-by: Zidan Wang <zidan.wang@freescale.com>
(cherry picked from commit fad2b9c1285a34eab0dec9f90edc06ec94e70a61)
Diffstat (limited to 'drivers/mfd')
| -rw-r--r-- | drivers/mfd/mxc-hdmi-core.c | 5 |
1 files changed, 4 insertions, 1 deletions
diff --git a/drivers/mfd/mxc-hdmi-core.c b/drivers/mfd/mxc-hdmi-core.c index ca79e7e4325a..a67040661b4a 100644 --- a/drivers/mfd/mxc-hdmi-core.c +++ b/drivers/mfd/mxc-hdmi-core.c @@ -153,9 +153,12 @@ int mxc_hdmi_register_audio(struct snd_pcm_substream *substream) unsigned long flags, flags1; int ret = 0; + if (!substream) + return -EINVAL; + snd_pcm_stream_lock_irqsave(substream, flags); - if (substream && check_hdmi_state()) { + if (check_hdmi_state()) { spin_lock_irqsave(&hdmi_audio_lock, flags1); if (hdmi_audio_stream_playback) { pr_err("%s unconsist hdmi auido stream!\n", __func__); |