diff options
| author | Xiaomeng Tong <xiam0nd.tong@gmail.com> | 2022-03-27 10:13:00 +0200 |
|---|---|---|
| committer | Mark Brown <broonie@kernel.org> | 2022-04-04 15:25:37 +0200 |
| commit | acc72863e0f11cd0bedc888b663700229f9ba5ff (patch) | |
| tree | ebcaaa7a5673647c1f85aab380f1318903a80654 /lib/crc16.c | |
| parent | ASoC: topology: Correct error handling in soc_tplg_dapm_widget_create() (diff) | |
| download | linux-acc72863e0f11cd0bedc888b663700229f9ba5ff.tar.xz linux-acc72863e0f11cd0bedc888b663700229f9ba5ff.zip | |
codecs: rt5682s: fix an incorrect NULL check on list iterator
The bug is here:
if (!dai) {
The list iterator value 'dai' will *always* be set and non-NULL
by for_each_component_dais(), so it is incorrect to assume that
the iterator value will be NULL if the list is empty or no element
is found (In fact, it will be a bogus pointer to an invalid struct
object containing the HEAD). Otherwise it will bypass the check
'if (!dai) {' (never call dev_err() and never return -ENODEV;)
and lead to invalid memory access lately when calling
'rt5682s_set_bclk1_ratio(dai, factor);'.
To fix the bug, just return rt5682s_set_bclk1_ratio(dai, factor);
when found the 'dai', otherwise dev_err() and return -ENODEV;
Cc: stable@vger.kernel.org
Fixes: bdd229ab26be9 ("ASoC: rt5682s: Add driver for ALC5682I-VS codec")
Signed-off-by: Xiaomeng Tong <xiam0nd.tong@gmail.com>
Link: https://lore.kernel.org/r/20220327081300.12962-1-xiam0nd.tong@gmail.com
Signed-off-by: Mark Brown <broonie@kernel.org>
Diffstat (limited to 'lib/crc16.c')
0 files changed, 0 insertions, 0 deletions