[FFmpeg-devel] [PATCH 3/3] Add assert that the avcodec lock is held when initializing static VLC tables.

[Michael Niedermayer]

On Tue, Nov 27, 2012 at 09:59:32PM +0100, Reimar Döffinger wrote:
> Signed-off-by: Reimar Döffinger <Reimar.Doeffinger at gmx.de>
> ---
>  libavcodec/bitstream.c |    2 ++
>  1 file changed, 2 insertions(+)
> 
> diff --git a/libavcodec/bitstream.c b/libavcodec/bitstream.c
> index ce83ee0..e986300 100644
> --- a/libavcodec/bitstream.c
> +++ b/libavcodec/bitstream.c
> @@ -30,6 +30,7 @@
>  
>  #include "libavutil/avassert.h"
>  #include "avcodec.h"
> +#include "internal.h"
>  #include "mathops.h"
>  #include "get_bits.h"
>  #include "put_bits.h"
> @@ -275,6 +276,7 @@ int ff_init_vlc_sparse(VLC *vlc, int nb_bits, int nb_codes,
>          }else if(vlc->table_size){
>              abort(); // fatal error, we are called on a partially initialized table
>          }
> +        av_assert0(ff_avcodec_locked);

this patch should be unneeded now and static vlc init be thread safe

[...]
-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

Republics decline into democracies and democracies degenerate into
despotisms. -- Aristotle
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 198 bytes
Desc: Digital signature
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20121206/3196d173/attachment.asc>