diff options
author | Geert Uytterhoeven <geert+renesas@glider.be> | 2015-01-13 21:03:37 +0100 |
---|---|---|
committer | Mark Brown <broonie@kernel.org> | 2015-01-14 17:16:37 +0000 |
commit | 7ddfdb5c5a5b51bdd2cb749d8341d763b079d520 (patch) | |
tree | 1bf9ac1cc99728ca42900fffe5b7d50daabd95ad /sound/soc/rockchip/rockchip_i2s.c | |
parent | 97bf6af1f928216fd6c5a66e8a57bfa95a659672 (diff) | |
download | blackbird-op-linux-7ddfdb5c5a5b51bdd2cb749d8341d763b079d520.tar.gz blackbird-op-linux-7ddfdb5c5a5b51bdd2cb749d8341d763b079d520.zip |
ASoC: simple-card: Fix crash in asoc_simple_card_unref()
If asoc_simple_card_probe() fails, asoc_simple_card_unref() may be
called before dev_set_drvdata(), causing a NULL pointer dereference in
asoc_simple_card_unref():
Unable to handle kernel NULL pointer dereference at virtual address 000000d4
...
PC is at asoc_simple_card_unref+0x14/0x48
LR is at asoc_simple_card_probe+0x3d4/0x40c
This typically happens because asoc_simple_card_parse_of() returns
-EPROBE_DEFER, but other failure modes are possible.
devm_snd_soc_register_card()/snd_soc_register_card() may fail either
before or after dev_set_drvdata().
Pass a snd_soc_card pointer instead of a platform_device pointer to
asoc_simple_card_unref() to fix this.
Note that if CONFIG_OF_DYNAMIC=n, of_node_put() is a dummy, and gcc may
optimize away the loop over card->dai_link, never actually dereferencing
card, and thus avoiding the crash...
Signed-off-by: Geert Uytterhoeven <geert+renesas@glider.be>
Fixes: e512e001dafa54e5 ("ASoC: simple-card: Fix the reference count of device nodes")
Signed-off-by: Mark Brown <broonie@kernel.org>
Cc: stable@vger.kernel.org
Diffstat (limited to 'sound/soc/rockchip/rockchip_i2s.c')
0 files changed, 0 insertions, 0 deletions