From ffaa3171985fb2a57d5eeb67d89e93c2c5a5f7fe Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?R=C3=A9mi=20Denis-Courmont?= Date: Wed, 16 Nov 2016 23:12:27 +0200 Subject: [PATCH] playlist: playlist_RecursiveNodeSort() asserts the playlist lock As any function that requires a valid playlist item as parameter, the function can only make sense with the lock already held by the caller. --- src/playlist/sort.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/src/playlist/sort.c b/src/playlist/sort.c index 1b6d1b6807..588bd35ab5 100644 --- a/src/playlist/sort.c +++ b/src/playlist/sort.c @@ -193,6 +193,8 @@ static int recursiveNodeSort( playlist_t *p_playlist, playlist_item_t *p_node, int playlist_RecursiveNodeSort( playlist_t *p_playlist, playlist_item_t *p_node, int i_mode, int i_type ) { + PL_ASSERT_LOCKED; + /* Ask the playlist to reset as we are changing the order */ pl_priv(p_playlist)->b_reset_currently_playing = true; -- GitLab