diff options
author | Takashi Iwai <tiwai@suse.de> | 2021-05-17 15:15:39 +0200 |
---|---|---|
committer | Takashi Iwai <tiwai@suse.de> | 2021-05-17 16:04:22 +0200 |
commit | a11aa8537e13dd1082c85b102b98afc2a156a815 (patch) | |
tree | 73c9d400c2dbd1679f310f6af58412525c8afbe3 | |
parent | ALSA: usx2y: Fix potential leaks of uninitialized memory (diff) | |
download | linux-a11aa8537e13dd1082c85b102b98afc2a156a815.tar.xz linux-a11aa8537e13dd1082c85b102b98afc2a156a815.zip |
ALSA: usx2y: Avoid self-killing
The initialization os usx2y driver is multi-staged, and the PCM and
other device creations are done after the DSP is loaded and
initialized. Upon the initialization, when an error happens, the
driver tries to call snd_card_free(). But this is dangerous, and in
general, the driver cannot kill itself during its operation.
Hence better to drop the snd_card_free() call from there.
Link: https://lore.kernel.org/r/20210517131545.27252-6-tiwai@suse.de
Signed-off-by: Takashi Iwai <tiwai@suse.de>
-rw-r--r-- | sound/usb/usx2y/usX2Yhwdep.c | 1 |
1 files changed, 0 insertions, 1 deletions
diff --git a/sound/usb/usx2y/usX2Yhwdep.c b/sound/usb/usx2y/usX2Yhwdep.c index 0ed50be89271..ec7e3beed4f9 100644 --- a/sound/usb/usx2y/usX2Yhwdep.c +++ b/sound/usb/usx2y/usX2Yhwdep.c @@ -220,7 +220,6 @@ static int snd_usx2y_hwdep_dsp_load(struct snd_hwdep *hw, err = usx2y_create_alsa_devices(hw->card); if (err) { snd_printk(KERN_ERR "usx2y_create_alsa_devices error %i\n", err); - snd_card_free(hw->card); return err; } priv->chip_status |= USX2Y_STAT_CHIP_INIT; |