]> git.baikalelectronics.ru Git - kernel.git/commit
ALSA: core: Fix card races between register and disconnect
authorTakashi Iwai <tiwai@suse.de>
Tue, 16 Apr 2019 15:06:33 +0000 (17:06 +0200)
committerTakashi Iwai <tiwai@suse.de>
Tue, 16 Apr 2019 15:06:33 +0000 (17:06 +0200)
commitc5acaa8faa28bce2678a84b0a04d4da5c2c6342e
treeeeffa65a589bb0dc6316eca30f1d9055446db4e2
parent829a4a478e1752c7d3c2deaaee86fadeb8db10e8
ALSA: core: Fix card races between register and disconnect

There is a small race window in the card disconnection code that
allows the registration of another card with the very same card id.
This leads to a warning in procfs creation as caught by syzkaller.

The problem is that we delete snd_cards and snd_cards_lock entries at
the very beginning of the disconnection procedure.  This makes the
slot available to be assigned for another card object while the
disconnection procedure is being processed.  Then it becomes possible
to issue a procfs registration with the existing file name although we
check the conflict beforehand.

The fix is simply to move the snd_cards and snd_cards_lock clearances
at the end of the disconnection procedure.  The references to these
entries are merely either from the global proc files like
/proc/asound/cards or from the card registration / disconnection, so
it should be fine to shift at the very end.

Reported-by: syzbot+48df349490c36f9f54ab@syzkaller.appspotmail.com
Cc: <stable@vger.kernel.org>
Signed-off-by: Takashi Iwai <tiwai@suse.de>
sound/core/init.c