mirror of
https://git.kernel.org/pub/scm/linux/kernel/git/stable/linux.git
synced 2024-09-29 22:02:02 +00:00
staging: greybus: codecs: fix type confusion of list iterator variable
[ Upstream commit84ef256550
] If the list does not exit early then data == NULL and 'module' does not point to a valid list element. Using 'module' in such a case is not valid and was therefore removed. Fixes:6dd67645f2
("greybus: audio: Use single codec driver registration") Reviewed-by: Dan Carpenter <dan.carpenter@oracle.com> Reviewed-by: Vaibhav Agarwal <vaibhav.sr@gmail.com> Reviewed-by: Mark Greer <mgreer@animalcreek.com> Signed-off-by: Jakob Koschel <jakobkoschel@gmail.com> Link: https://lore.kernel.org/r/20220321123626.3068639-1-jakobkoschel@gmail.com Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org> Signed-off-by: Sasha Levin <sashal@kernel.org>
This commit is contained in:
parent
4063e2fecd
commit
61ca94fab3
1 changed files with 2 additions and 2 deletions
|
@ -618,8 +618,8 @@ static int gbcodec_mute_stream(struct snd_soc_dai *dai, int mute, int stream)
|
||||||
break;
|
break;
|
||||||
}
|
}
|
||||||
if (!data) {
|
if (!data) {
|
||||||
dev_err(dai->dev, "%s:%s DATA connection missing\n",
|
dev_err(dai->dev, "%s DATA connection missing\n",
|
||||||
dai->name, module->name);
|
dai->name);
|
||||||
mutex_unlock(&codec->lock);
|
mutex_unlock(&codec->lock);
|
||||||
return -ENODEV;
|
return -ENODEV;
|
||||||
}
|
}
|
||||||
|
|
Loading…
Reference in a new issue