Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit 46a8e16f authored by Pierre-Louis Bossart's avatar Pierre-Louis Bossart Committed by Greg Kroah-Hartman
Browse files

ALSA: hda: hdac_stream: fix potential locking issue in snd_hdac_stream_assign() 



commit 1465d06a6d8580e73ae65f8590392df58c5ed2fd upstream.

The fields 'opened', 'running', 'assigned_key' are all protected by a
spinlock, but the spinlock is not taken when looking for a
stream. This can result in a possible race between assign() and
release().

Fix by taking the spinlock before walking through the bus stream list.

Signed-off-by: default avatarPierre-Louis Bossart <pierre-louis.bossart@linux.intel.com>
Link: https://lore.kernel.org/r/20210924192417.169243-2-pierre-louis.bossart@linux.intel.com


Signed-off-by: default avatarTakashi Iwai <tiwai@suse.de>
Cc: Scott Bruce <smbruce@gmail.com>
Signed-off-by: default avatarGreg Kroah-Hartman <gregkh@linuxfoundation.org>
parent 29338573

sound/hda/hdac_stream.c

+2 −2
Original line number Diff line number Diff line
@@ -289,6 +289,7 @@ struct hdac_stream *snd_hdac_stream_assign(struct hdac_bus *bus, 
	int key = (substream->pcm->device << 16) | (substream->number << 2) |

		(substream->stream + 1);



	spin_lock_irq(&bus->reg_lock);

	list_for_each_entry(azx_dev, &bus->stream_list, list) {

		if (azx_dev->direction != substream->stream)

			continue;
@@ -302,13 +303,12 @@ struct hdac_stream *snd_hdac_stream_assign(struct hdac_bus *bus, 
			res = azx_dev;

	}

	if (res) {

		spin_lock_irq(&bus->reg_lock);

		res->opened = 1;

		res->running = 0;

		res->assigned_key = key;

		res->substream = substream;

		spin_unlock_irq(&bus->reg_lock);

	}

	spin_unlock_irq(&bus->reg_lock);

	return res;

}

EXPORT_SYMBOL_GPL(snd_hdac_stream_assign);