diff options
| author | Cezary Rojewski <cezary.rojewski@intel.com> | 2020-03-12 15:00:58 +0300 |
|---|---|---|
| committer | Mark Brown <broonie@kernel.org> | 2020-03-12 16:12:35 +0300 |
| commit | 16dcefc23eefda57ee705dcffba67f1bd42c01bc (patch) | |
| tree | e5d16f3397e1be75a93a461a69ec0f1a1c0c3cda /lib/test-string_helpers.c | |
| parent | a5107b1a0993ff035dc8a1cf331da7bf961e4cbb (diff) | |
| download | linux-16dcefc23eefda57ee705dcffba67f1bd42c01bc.tar.xz | |
ASoC: SOF: Intel: Fix stream cleanup on hw free
Field "substream" gets assigned during stream setup in
hda_dsp_pcm_hw_params() but it is never cleared afterwards during
cleanup procedure. Now, any non-pcm operation e.g.: compress can
mistakenly make use of that pointer as it's bypassing all
"if (s->substream)" checks.
Nulling the pointer during hw_free operation ensures no wild pointers
are left behind.
Fixes: cdae3b9a47aa ("ASoC: SOF: Intel: Add Intel specific HDA PCM operations")
Signed-off-by: Cezary Rojewski <cezary.rojewski@intel.com>
Acked-by: Pierre-Louis Bossart <pierre-louis.bossart@linux.intel.com>
Link: https://lore.kernel.org/r/20200312120058.15057-1-cezary.rojewski@intel.com
Signed-off-by: Mark Brown <broonie@kernel.org>
Diffstat (limited to 'lib/test-string_helpers.c')
0 files changed, 0 insertions, 0 deletions