On Wed, Mar 11, 2015 at 9:34 PM, Luca Barbato <[email protected]> wrote: > On 11/03/15 21:43, Vittorio Giovara wrote: >> >> --- >> Updated as requested. >> Vittorio >> >> libavcodec/utils.c | 5 ++++- >> 1 file changed, 4 insertions(+), 1 deletion(-) >> >> diff --git a/libavcodec/utils.c b/libavcodec/utils.c >> index d568cbb..7b169ff 100644 >> --- a/libavcodec/utils.c >> +++ b/libavcodec/utils.c >> @@ -1050,7 +1050,10 @@ int attribute_align_arg >> avcodec_open2(AVCodecContext *avctx, const AVCodec *code >> >> entangled_thread_counter++; >> if (entangled_thread_counter != 1) { >> - av_log(avctx, AV_LOG_ERROR, "insufficient thread locking around >> avcodec_open/close()\n"); >> + av_log(avctx, AV_LOG_ERROR, >> + "Insufficient thread locking. At least %d threads are " >> + "calling avcodec_open2() at the same time right now.\n", >> + entangled_thread_counter); >> ret = -1; >> goto end; >> } >> > > Set Ok if coverity and fate are happy.
Oracle looks green and no new issues from Coverity. So if no further comments I'll push this code sometime tomorrow. Thanks to all the people who reviewed and commented the code. -- Vittorio _______________________________________________ libav-devel mailing list [email protected] https://lists.libav.org/mailman/listinfo/libav-devel
