diff options
| author | Lucas Tanure <tanureal@opensource.cirrus.com> | 2021-07-28 11:44:16 +0100 |
|---|---|---|
| committer | Mark Brown <broonie@kernel.org> | 2021-07-28 16:39:15 +0100 |
| commit | acbf58e530416e167c3b323111f4013d9f2b0a7d (patch) | |
| tree | 0bb93f7a747a0eb196a5e07aaed68691eecb57b0 | |
| parent | 31428c78748cafdd9352e1f622eb89bf453d9700 (diff) | |
| download | linux-acbf58e530416e167c3b323111f4013d9f2b0a7d.tar.bz2 | |
ASoC: wm_adsp: Let soc_cleanup_component_debugfs remove debugfs
soc_cleanup_component_debugfs will debugfs_remove_recursive
the component->debugfs_root, so adsp doesn't need to also
remove the same entry.
By doing that adsp also creates a race with core component,
which causes a NULL pointer dereference
Signed-off-by: Lucas Tanure <tanureal@opensource.cirrus.com>
Link: https://lore.kernel.org/r/20210728104416.636591-1-tanureal@opensource.cirrus.com
Signed-off-by: Mark Brown <broonie@kernel.org>
| -rw-r--r-- | sound/soc/codecs/wm_adsp.c | 1 |
1 files changed, 0 insertions, 1 deletions
diff --git a/sound/soc/codecs/wm_adsp.c b/sound/soc/codecs/wm_adsp.c index 549d98241dae..fe15cbc7bcaf 100644 --- a/sound/soc/codecs/wm_adsp.c +++ b/sound/soc/codecs/wm_adsp.c @@ -747,7 +747,6 @@ static void wm_adsp2_init_debugfs(struct wm_adsp *dsp, static void wm_adsp2_cleanup_debugfs(struct wm_adsp *dsp) { wm_adsp_debugfs_clear(dsp); - debugfs_remove_recursive(dsp->debugfs_root); } #else static inline void wm_adsp2_init_debugfs(struct wm_adsp *dsp, |