[alsa-devel] [PATCH] ASoC: dapm: Fix empty list check in dapm_new_mux()
Stephen Warren
swarren at wwwdotorg.org
Thu Aug 1 19:20:34 CEST 2013
On 08/01/2013 10:30 AM, Lars-Peter Clausen wrote:
> list_first_entry() will always return a valid pointer, even if the list is
> empty. So the check whether path is NULL will always be false. So we end up
> calling dapm_create_or_share_mixmux_kcontrol() with a path struct that points
> right in the middle of the widget struct and by trying to modify the path the
> widgets memory will become corrupted. Fix this by using list_emtpy() to check if
> the widget doesn't have any paths.
>
> Signed-off-by: Lars-Peter Clausen <lars at metafoo.de>
>
> ---
> This is the real fix for the rt5640 crash.
Tested-by: Stephen Warren <swarren at nvidia.com>
More information about the Alsa-devel
mailing list