ALSA: usb-audio: Fix race against the error recovery URB submission
authorTakashi Iwai <tiwai@suse.de>
Fri, 10 Jul 2020 16:06:56 +0000 (18:06 +0200)
committerTakashi Iwai <tiwai@suse.de>
Fri, 10 Jul 2020 16:07:25 +0000 (18:07 +0200)
commit9b7e5208a941e2e491a83eb5fa83d889e888fa2f
tree59c0a0415ecbeaabd3c8c86769110a06dd94f301
parent68359a1ad8447c99732ebeab8c169bfed543667a
ALSA: usb-audio: Fix race against the error recovery URB submission

USB MIDI driver has an error recovery mechanism to resubmit the URB in
the delayed timer handler, and this may race with the standard start /
stop operations.  Although both start and stop operations themselves
don't race with each other due to the umidi->mutex protection, but
this isn't applied to the timer handler.

For fixing this potential race, the following changes are applied:

- Since the timer handler can't use the mutex, we apply the
  umidi->disc_lock protection at each input stream URB submission;
  this also needs to change the GFP flag to GFP_ATOMIC
- Add a check of the URB refcount and skip if already submitted
- Move the timer cancel call at disconnection to the beginning of the
  procedure; this assures the in-flight timer handler is gone properly
  before killing all pending URBs

Reported-by: syzbot+0f4ecfe6a2c322c81728@syzkaller.appspotmail.com
Reported-by: syzbot+5f1d24c49c1d2c427497@syzkaller.appspotmail.com
Cc: <stable@vger.kernel.org>
Link: https://lore.kernel.org/r/20200710160656.16819-1-tiwai@suse.de
Signed-off-by: Takashi Iwai <tiwai@suse.de>
sound/usb/midi.c