]> git.baikalelectronics.ru Git - kernel.git/commit
ALSA: usb-audio: Fix races at disconnection and PCM closing
authorTakashi Iwai <tiwai@suse.de>
Wed, 25 Jun 2014 12:24:47 +0000 (14:24 +0200)
committerTakashi Iwai <tiwai@suse.de>
Thu, 26 Jun 2014 08:33:35 +0000 (10:33 +0200)
commitfae88176d8c4870ea8745623f1115cc9dd196d28
treee171b5669ae5ebc62c8167e41f9d849382b3bd59
parent5e5ee1f47d1ef728d8559c38ba4f0149722a20ae
ALSA: usb-audio: Fix races at disconnection and PCM closing

When a USB-audio device is disconnected while PCM is still running, we
still see some race: the disconnect callback calls
snd_usb_endpoint_free() that calls release_urbs() and then kfree()
while a PCM stream would be closed at the same time and calls
stop_endpoints() that leads to wait_clear_urbs().  That is, the EP
object might be deallocated while a PCM stream is syncing with
wait_clear_urbs() with the same EP.

Basically calling multiple wait_clear_urbs() would work fine, also
calling wait_clear_urbs() and release_urbs() would work, too, as
wait_clear_urbs() just reads some fields in ep.  The problem is the
succeeding kfree() in snd_pcm_endpoint_free().

This patch moves out the EP deallocation into the later point, the
destructor callback.  At this stage, all PCMs must have been already
closed, so it's safe to free the objects.

Reported-by: Alan Stern <stern@rowland.harvard.edu>
Cc: <stable@vger.kernel.org>
Signed-off-by: Takashi Iwai <tiwai@suse.de>
sound/usb/card.c
sound/usb/endpoint.c
sound/usb/endpoint.h