diff options
| author | Derek Buitenhuis <derek.buitenhuis@gmail.com> | 2013-10-10 11:05:40 -0400 |
|---|---|---|
| committer | Derek Buitenhuis <derek.buitenhuis@gmail.com> | 2013-10-13 05:50:00 -0400 |
| commit | eb90a2091ffb94d8c29aaa5ff50f4192520254fc (patch) | |
| tree | 86a3e267527f39fca087c8c51242c2acaef933a2 | |
| parent | c9ef6b09326a24010bf86d6b0d19cfa42df4d546 (diff) | |
| download | ffmpeg-eb90a2091ffb94d8c29aaa5ff50f4192520254fc.tar.gz | |
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 <derek.buitenhuis@gmail.com>
| -rw-r--r-- | libavcodec/pthread.c | 3 | ||||
| -rw-r--r-- | 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) { |