]> git.baikalelectronics.ru Git - kernel.git/commit
ALSA: aloop: Sync stale timer before release
authorTakashi Iwai <tiwai@suse.de>
Thu, 22 Mar 2018 07:56:06 +0000 (08:56 +0100)
committerTakashi Iwai <tiwai@suse.de>
Thu, 22 Mar 2018 09:34:12 +0000 (10:34 +0100)
commit2cd6f1ddb1838f2f109681acf388c78a171d94a0
treeb499d7415039119c5bc6441d9ac2ed416fba70b8
parent7183b388ec5a62a19022f6c6933c81c57ce0fce3
ALSA: aloop: Sync stale timer before release

The aloop driver tries to stop the pending timer via timer_del() in
the trigger callback and in the close callback.  The former is
correct, as it's an atomic operation, while the latter expects that
the timer gets really removed and proceeds the resource releases after
that.  But timer_del() doesn't synchronize, hence the running timer
may still access the released resources.

A similar situation can be also seen in the prepare callback after
trigger(STOP) where the prepare tries to re-initialize the things
while a timer is still running.

The problems like the above are seen indirectly in some syzkaller
reports (although it's not 100% clear whether this is the only cause,
as the race condition is quite narrow and not always easy to
trigger).

For addressing these issues, this patch adds the explicit alls of
timer_del_sync() in some places, so that the pending timer is properly
killed / synced.

Cc: <stable@vger.kernel.org>
Signed-off-by: Takashi Iwai <tiwai@suse.de>
sound/drivers/aloop.c