From: Takashi Iwai Date: Mon, 5 Mar 2018 21:00:55 +0000 (+0100) Subject: ALSA: seq: Don't allow resizing pool in use X-Git-Tag: v4.1.51~35 X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=282e27a729a6f766354e39a35f4e77f615a8e2ac;p=platform%2Fkernel%2Flinux-exynos.git ALSA: seq: Don't allow resizing pool in use [ Upstream commit d85739367c6d56e475c281945c68fdb05ca74b4c ] This is a fix for a (sort of) fallout in the recent commit d15d662e89fc ("ALSA: seq: Fix racy pool initializations") for CVE-2018-1000004. As the pool resize deletes the existing cells, it may lead to a race when another thread is writing concurrently, eventually resulting a UAF. A simple workaround is not to allow the pool resizing when the pool is in use. It's an invalid behavior in anyway. Fixes: d15d662e89fc ("ALSA: seq: Fix racy pool initializations") Reported-by: 范龙飞 Reported-by: Nicolai Stange Cc: Signed-off-by: Takashi Iwai Signed-off-by: Sasha Levin --- diff --git a/sound/core/seq/seq_clientmgr.c b/sound/core/seq/seq_clientmgr.c index b9ce5da25938..916c9c0069ed 100644 --- a/sound/core/seq/seq_clientmgr.c +++ b/sound/core/seq/seq_clientmgr.c @@ -1924,6 +1924,9 @@ static int snd_seq_ioctl_set_client_pool(struct snd_seq_client *client, (! snd_seq_write_pool_allocated(client) || info.output_pool != client->pool->size)) { if (snd_seq_write_pool_allocated(client)) { + /* is the pool in use? */ + if (atomic_read(&client->pool->counter)) + return -EBUSY; /* remove all existing cells */ snd_seq_pool_mark_closing(client->pool); snd_seq_queue_client_leave_cells(client->number);