ALSA: hda: Fix potential race at unregistration and unsol events
authorTakashi Iwai <tiwai@suse.de>
Mon, 19 Jun 2017 15:49:48 +0000 (17:49 +0200)
committerTakashi Iwai <tiwai@suse.de>
Tue, 20 Jun 2017 05:53:57 +0000 (07:53 +0200)
commiteb8d0eaaf84b0398533a7c091a0b65663f2fd7ea
treedfb85ea2e1ee4ff8134aef89b7ba933d9f2920d6
parenta4b4793f640b72af3e2bb2a1ad79725c103d5e40
ALSA: hda: Fix potential race at unregistration and unsol events

When the codec device is unregistered / freed, it may release the
resource while being used in an unsolicited event like the jack
detection work.  This leads to use-after-free.

The fix here is to unregister the device at first, i.e. removing the
codec from the list, then flushing the pending works to assure that
all unsol events are gone.  After this point, we're free from
accessing the codec via unsol events, thus can release the resources
gracefully.

The issue was spotted originally by Intel CI, but it couldn't be
reproduced reliably by its nature.  So let's hope this fix really
addresses the whole issues.

Bugzilla: https://bugzilla.kernel.org/show_bug.cgi?id=196045
Reported-by: Martin Peres <martin.peres@free.fr>
Signed-off-by: Takashi Iwai <tiwai@suse.de>
sound/hda/hdac_bus.c
sound/hda/hdac_device.c