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 /libavcodec | |
| parent | c9ef6b09326a24010bf86d6b0d19cfa42df4d546 (diff) | |
| download | ffmpeg-streaming-eb90a2091ffb94d8c29aaa5ff50f4192520254fc.zip ffmpeg-streaming-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>
Diffstat (limited to 'libavcodec')
| -rw-r--r-- | libavcodec/pthread.c | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/libavcodec/pthread.c b/libavcodec/pthread.c index f4795f3..b0d9e27 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) { |
