diff options
author | Takashi Iwai <tiwai@suse.de> | 2022-05-25 15:12:03 +0200 |
---|---|---|
committer | Takashi Iwai <tiwai@suse.de> | 2022-05-25 15:13:29 +0200 |
commit | 0125de38122f0f66bf61336158d12a1aabfe6425 (patch) | |
tree | 26e8bec01e7245752d55e452d6a7a501c42619b9 /sound/usb | |
parent | 4c691a287d4ee0c308708c1d6f9e0cc7513463f8 (diff) | |
download | lwn-0125de38122f0f66bf61336158d12a1aabfe6425.tar.gz lwn-0125de38122f0f66bf61336158d12a1aabfe6425.zip |
ALSA: usb-audio: Cancel pending work at closing a MIDI substream
At closing a USB MIDI output substream, there might be still a pending
work, which would eventually access the rawmidi runtime object that is
being released. For fixing the race, make sure to cancel the pending
work at closing.
Reported-by: syzbot+6912c9592caca7ca0e7d@syzkaller.appspotmail.com
Cc: <stable@vger.kernel.org>
Link: https://lore.kernel.org/r/000000000000e7e75005dfd07cf6@google.com
Link: https://lore.kernel.org/r/20220525131203.11299-1-tiwai@suse.de
Signed-off-by: Takashi Iwai <tiwai@suse.de>
Diffstat (limited to 'sound/usb')
-rw-r--r-- | sound/usb/midi.c | 3 |
1 files changed, 3 insertions, 0 deletions
diff --git a/sound/usb/midi.c b/sound/usb/midi.c index 7c6ca2b433a5..344fbeadf161 100644 --- a/sound/usb/midi.c +++ b/sound/usb/midi.c @@ -1145,6 +1145,9 @@ static int snd_usbmidi_output_open(struct snd_rawmidi_substream *substream) static int snd_usbmidi_output_close(struct snd_rawmidi_substream *substream) { + struct usbmidi_out_port *port = substream->runtime->private_data; + + cancel_work_sync(&port->ep->work); return substream_open(substream, 0, 0); } |