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 /libavfilter/pthread.c | |
| parent | c9ef6b09326a24010bf86d6b0d19cfa42df4d546 (diff) | |
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>
Diffstat (limited to 'libavfilter/pthread.c')
| -rw-r--r-- | libavfilter/pthread.c | 3 |
1 files changed, 2 insertions, 1 deletions
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) { |