[vlc-devel] RE : [PATCH] -- skins2 (more corrections)

brezhoneg1 brezhoneg1 at yahoo.fr
Mon Mar 16 21:53:37 CET 2009

> I must say that...
>      if( !b_playlist_locked ) PL_UNLOCK;
> +
> +    if( b_playlist_locked ) PL_UNLOCK;
> ...that smells utmostly fishy to me.

The function playlist_CurrentInput( ) automatically holds the playlist.
Since this lock may already be held before entering the function, this
results in EDEADLCK(35).

Maybe a recursive lock would solve this issue ? 

What I did was to quickly overcome the problem by unlocking just before
and relocking just after, but I agree this should not be the long term


More information about the vlc-devel mailing list