From eb90a2091ffb94d8c29aaa5ff50f4192520254fc Mon Sep 17 00:00:00 2001 From: Derek Buitenhuis Date: Thu, 10 Oct 2013 11:05:40 -0400 Subject: [PATCH] pthread: Fix deadlock during thread initialization Sometimes, if pthread_create() failed, then pthread_cond_wait() could accidentally be called in the worker threads after the uninit function had already called pthread_cond_broadcast(), leading to a deadlock. Don't call pthread_cond_wait() if c->done is set. Signed-off-by: Derek Buitenhuis --- libavcodec/pthread.c | 3 ++- libavfilter/pthread.c | 3 ++- 2 files changed, 4 insertions(+), 2 deletions(-) diff --git a/libavcodec/pthread.c b/libavcodec/pthread.c index f4795f3444..b0d9e27aa9 100644 --- a/libavcodec/pthread.c +++ b/libavcodec/pthread.c @@ -152,7 +152,8 @@ static void* attribute_align_arg worker(void *v) if (c->current_job == thread_count + c->job_count) pthread_cond_signal(&c->last_job_cond); - pthread_cond_wait(&c->current_job_cond, &c->current_job_lock); + if (!c->done) + pthread_cond_wait(&c->current_job_cond, &c->current_job_lock); our_job = self_id; if (c->done) { diff --git a/libavfilter/pthread.c b/libavfilter/pthread.c index a4f45c82cd..17e8e7b20f 100644 --- a/libavfilter/pthread.c +++ b/libavfilter/pthread.c @@ -73,7 +73,8 @@ static void* attribute_align_arg worker(void *v) if (c->current_job == nb_threads + c->nb_jobs) pthread_cond_signal(&c->last_job_cond); - pthread_cond_wait(&c->current_job_cond, &c->current_job_lock); + if (!c->done) + pthread_cond_wait(&c->current_job_cond, &c->current_job_lock); our_job = self_id; if (c->done) {