player_control: moved pc.next_song assertion into lock

The assertion shouldn't access player_control.next_song without locking it.

player_control: moved pc.next_song assertion into lock
f421c8a1 · Max Kellermann · d06f8baf · f421c8a1
Commit f421c8a1 authored Nov 03, 2009 by Max Kellermann
Hide whitespace changes
Inline Side-by-side

Showing with 2 additions and 1 deletion

player_control.c src/player_control.c +2 -1

No files found.
--- a/src/player_control.c
+++ b/src/player_control.c
@@ -255,9 +255,10 @@ void
 pc_enqueue_song(struct song *song)
 {
 	assert(song != NULL);
-	assert(pc.next_song == NULL);

 	player_lock();
+	assert(pc.next_song == NULL);
+
 	pc.next_song = song;
 	player_command_locked(PLAYER_COMMAND_QUEUE);
 	player_unlock();