From 56b8081af6a623a96ef24fff027b0b40f478cbf2 Mon Sep 17 00:00:00 2001 From: Max Kellermann Date: Tue, 3 Nov 2009 21:23:48 +0100 Subject: player_thread: lock player in player_check_decoder_startup() Don't access attributes without the lock. --- src/player_thread.c | 8 ++++++++ 1 file changed, 8 insertions(+) diff --git a/src/player_thread.c b/src/player_thread.c index b6599ed0d..012cc3964 100644 --- a/src/player_thread.c +++ b/src/player_thread.c @@ -236,8 +236,10 @@ player_check_decoder_startup(struct player *player) /* the decoder failed */ decoder_unlock(dc); + player_lock(); pc.errored_song = dc->song; pc.error = PLAYER_ERROR_FILE; + player_unlock(); return false; } else if (!decoder_is_starting(dc)) { @@ -251,8 +253,11 @@ player_check_decoder_startup(struct player *player) all chunks yet - wait for that */ return true; + player_lock(); pc.total_time = dc->total_time; pc.audio_format = dc->in_audio_format; + player_unlock(); + player->play_audio_format = dc->out_audio_format; player->decoder_starting = false; @@ -264,11 +269,14 @@ player_check_decoder_startup(struct player *player) "while playing \"%s\"", uri); g_free(uri); + player_lock(); pc.error = PLAYER_ERROR_AUDIO; /* pause: the user may resume playback as soon as an audio output becomes available */ pc.state = PLAYER_STATE_PAUSE; + player_unlock(); + player->paused = true; return true; } -- cgit v1.2.3