[vlc-devel] [PATCH] -- vout_subpicture deadlock issue
rem at videolan.org
Sun Dec 7 20:45:32 CET 2008
Le dimanche 7 décembre 2008 17:06:21 brezhoneg1, vous avez écrit :
> This patch corrects a bug which was made visible thanks to the following
> commit :
> bb0d8ab4701575 (a vlc_assert_locked( &p_sys->lock ) had been added).
> the "SubFilterAllocationClean" function obviously does not hold a lock
> and yet set the b_lock parameter of SpuCleanChannel to true. This was
> tested in at least two situations:
> - see trac item https://trac.videolan.org/vlc/ticket/2281
> - setting a new media for a media_player (libvlc api) now raises this
> deadlock error too.
> Setting this parameter to false solves these two problems.
> (Yet, are there situations where it does hold the lock?, if so, deeper
> debugging work has to be done )
Seemingly not, but I really am not an expert on this stuff.
That being said, you should really the parameter completely as your patch
makes it always false.
More information about the vlc-devel